feat(ios): Implement property 'quality' in captureImage() (CB-11630)

[ryanwilliams83]

Platforms affected

iOS

What does this PR do?

Allows developers to set JPEG compression quality

What testing has been done on this change?

captureImage()
'quality' omitted / 0 / 0.0 / 0.1 / 0.3 / 0.5 / 0.96 / 0.99 / 1 / 100

Checklist

• ICLA has been signed and submitted to [email protected].
• Reported an issue in the JIRA database
• Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
• Added automated test coverage as appropriate for this change.

[timbru31]

Old but gold! LGTM 👍

[janpio]

Is there anything comparable in one of our other plugins?
Is there an equivalent feature possible for Android?
What happens on Android when this param is supplied, which is not support there (yet)?
Any way to test this via automated tests?

[timbru31]

Is there anything comparable in one of our other plugins?

What do you mean?

Is there an equivalent feature possible for Android

Haven’t searched for such a feature yet. And besides android the other plugins would need to be check, too.

What happens on Android when this param is supplied, which is not support there (yet)?

Hmm 🤔 The android thing could indeed lead to issues, since quality is parsed from the JSON as an integer: https://github.com/apache/cordova-plugin-media-capture/blob/master/src/android/PendingRequests.java#L191

…

Sent from my iPhone

On 7. May 2019, at 12:04 AM, Jan Piotrowski ***@***.***> wrote: Is there anything comparable in one of our other plugins? Is there an equivalent feature possible for Android? What happens on Android when this param is supplied, which is not support there (yet)? — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[janpio]

Is there anything comparable in one of our other plugins?
What do you mean?

This is not the only plugin taking pictures. Does e.g. camera plugin have similar functionality? If so, does the API match?