Do we know when the API rework is scheduled?
On Wed, Oct 24, 2012 at 1:59 PM, Filip Maj <f...@adobe.com> wrote: > Should we slate this for the API rework or get to it sooner? > > On 10/23/12 1:07 PM, "Tim Kim" <timki...@gmail.com> wrote: > >>+1 >> >>On 23 October 2012 11:55, Shazron <shaz...@gmail.com> wrote: >> >>> +1 >>> >>> On Tue, Oct 23, 2012 at 11:51 AM, Simon MacDonald >>> <simon.macdon...@gmail.com> wrote: >>> > +1 to adding error codes >>> > +1 to cancel being reported by the error callback >>> > >>> > You may want to check out what Capture uses for error codes. >>> > >>> > >>> >>>http://docs.phonegap.com/en/2.1.0/cordova_media_capture_capture.md.html#C >>>aptureError >>> > >>> > Simon Mac Donald >>> > http://hi.im/simonmacdonald >>> > >>> > >>> > On Tue, Oct 23, 2012 at 2:40 PM, Filip Maj <f...@adobe.com> wrote: >>> > >>> >> So, our camera API doesn't have error codes. Per the docs, it >>>returns a >>> >> string [1]. Gnarly. >>> >> >>> >> The strings are also all different across platforms. Android [2] and >>>iOS >>> >> [3] (search for "error"). >>> >> >>> >> For extra-good times, BlackBerry-webworks-java [4] does not ever >>>return >>> an >>> >> error, except in the case where the file is deemed "too big" [5]. >>>So, if >>> >> the camera app is closed, we don't return an error code back to the >>>user >>> >> for this platform, which is different from iOS + Android. >>> >> >>> >> BlackBerry-webworks-air registers JS-based callbacks using the BB >>>HTML5 >>> >> transport API [6]. Unfortunately I can't find documentation about >>>what >>> >> kind of parameters are passed into the error callbacks (anyone from >>>RIM >>> >> help me out?). Finally, if the camera is closed, no callback gets >>> invoked >>> >> either [7], which is at odds with Android + iOS. >>> >> >>> >> BlackBerry 10 implementation [8] no-ops the onCancel and onInvoke >>> >> callbacks as well, which again, is at odds. >>> >> >>> >> --- >>> >> >>> >> Proposal: add error codes. >>> >> >>> >> [1] >>> >> >>> >>>http://docs.phonegap.com/en/2.1.0/cordova_camera_camera.md.html#cameraErr >>>or >>> >> [2] >>> >> >>> >>>https://github.com/apache/incubator-cordova-android/blob/master/framework >>>/s >>> >> rc/org/apache/cordova/CameraLauncher.java#L352-L363 >>> >> [3] >>> >> >>> >>>https://github.com/apache/incubator-cordova-ios/blob/master/CordovaLib/Cl >>>as >>> >> ses/CDVCamera.m#L80 >>> >> [4] >>> >> >>> >>>https://github.com/apache/incubator-cordova-blackberry-webworks/blob/mast >>>er >>> >> /framework/ext/src/org/apache/cordova/camera/Camera.java >>> >> [5] >>> >> >>> >>>https://github.com/apache/incubator-cordova-blackberry-webworks/blob/mast >>>er >>> >> /framework/ext/src/org/apache/cordova/camera/Camera.java#L225 >>> >> [6] >>> >> >>> >>>https://github.com/apache/incubator-cordova-js/blob/master/lib/webworks/a >>>ir >>> >> /plugin/air/camera.js >>> >> [7] >>> >> >>> >>>https://github.com/apache/incubator-cordova-js/blob/master/lib/webworks/a >>>ir >>> >> /plugin/air/camera.js#L27 >>> >> [8] >>> >> >>> >>>https://github.com/apache/incubator-cordova-js/blob/master/lib/webworks/q >>>nx >>> >> /plugin/qnx/camera.js >>> >> >>> >> >>> >> >> >> >>-- >>Timothy Kim >
