Return-Path: X-Original-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 62766D4B1 for ; Wed, 24 Oct 2012 21:03:09 +0000 (UTC) Received: (qmail 87135 invoked by uid 500); 24 Oct 2012 21:03:09 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 87110 invoked by uid 500); 24 Oct 2012 21:03:09 -0000 Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: callback-dev@incubator.apache.org Delivered-To: mailing list callback-dev@incubator.apache.org Received: (qmail 87102 invoked by uid 99); 24 Oct 2012 21:03:09 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 24 Oct 2012 21:03:09 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of shazron@gmail.com designates 209.85.212.47 as permitted sender) Received: from [209.85.212.47] (HELO mail-vb0-f47.google.com) (209.85.212.47) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 24 Oct 2012 21:03:04 +0000 Received: by mail-vb0-f47.google.com with SMTP id ez10so1057814vbb.6 for ; Wed, 24 Oct 2012 14:02:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type; bh=ur2+IDgwE+kQ3w7glOr1L9/z70AWfcT8CQK57Zqi7w8=; b=prXrbhECU8JvuaaYT4GLeIQver06JrIDjikNwGHMFwcB9elT3G317YIDNAHGVSF0/z iXeArDnteReU6xistBTAQv9BgfMYnLnU+Y3qcI1zo9YfD6/w/vPSDewp+roFG/agutf1 eMq+CbuGr5mjTIy4SrI+kax6Qh5udcgy1aX3zPIiqpUoCnGN6liOduMOrbeGzBZxwoZz h/wt1nr8elljsAMyLa4KXPHj5+6cFFKelnnh472VXqawgGpOMfRY9iOs3LI8zgG4mjsr OdfLTfg8FsMuXmcdvhOHgej3f73ygaEASWpDC3jaX46BxAcOsXx25uyS9U4CrBGr2mod PDmw== Received: by 10.220.231.8 with SMTP id jo8mr10403722vcb.40.1351112564078; Wed, 24 Oct 2012 14:02:44 -0700 (PDT) MIME-Version: 1.0 Received: by 10.58.211.232 with HTTP; Wed, 24 Oct 2012 14:02:03 -0700 (PDT) In-Reply-To: References: From: Shazron Date: Wed, 24 Oct 2012 14:02:03 -0700 Message-ID: Subject: Re: Camera API error codes? To: callback-dev@incubator.apache.org Content-Type: text/plain; charset=ISO-8859-1 X-Virus-Checked: Checked by ClamAV on apache.org Do we know when the API rework is scheduled? On Wed, Oct 24, 2012 at 1:59 PM, Filip Maj wrote: > Should we slate this for the API rework or get to it sooner? > > On 10/23/12 1:07 PM, "Tim Kim" wrote: > >>+1 >> >>On 23 October 2012 11:55, Shazron wrote: >> >>> +1 >>> >>> On Tue, Oct 23, 2012 at 11:51 AM, Simon MacDonald >>> 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 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 >