Apache Cordova
  1. Apache Cordova
  2. CB-506

images taken via Camera.getPicture do not get deleted

    Details

      Description

      images taken with camera.getPicture are stored in the temporary directory. There is code in applicationWillTerminate to delete the temporary directory. However, in a multitasking app this method is never called. The contents of this directory do not get backed up, but it is also not getting deleted. Seems like the only solution is to require the developer to clean out the temp directory since they know when it is appropriate to do so. We can leave the code in applicationWillTerminate for apps which set UIApplicationExistsOnSuspend in the plist since it IS called.
      Could discuss adding a method to Camera to do clean up rather than requiring devs. to use the File api's. Will need to negotiate with other platforms to see if this makes sense. Will add a dependency in Camera to File Plugin.

        Activity

        Show
        Becky Gibson added a comment - see https://groups.google.com/forum/?fromgroups#!topic/phonegap/qmFChwiB0To
        Hide
        Anthony Keller added a comment -

        Would it be possible to parameterize where the image is stored from camera.getPicture - i.e. temporary or persistent storage? If persistent is selected the image is stored in a persistent folder; if temporary is selected the image is stored in the tmp folder.

        Show
        Anthony Keller added a comment - Would it be possible to parameterize where the image is stored from camera.getPicture - i.e. temporary or persistent storage? If persistent is selected the image is stored in a persistent folder; if temporary is selected the image is stored in the tmp folder.
        Hide
        Shazron Abdullah added a comment -

        Added all platforms to the discussion.

        All platforms need to discuss adding a clean() method or something to that effect to fix this. Note: devs can use the File APIs to clean this up, but it will be better if we provide a convenience function, if not in native, then in JS.

        Show
        Shazron Abdullah added a comment - Added all platforms to the discussion. All platforms need to discuss adding a clean() method or something to that effect to fix this. Note: devs can use the File APIs to clean this up, but it will be better if we provide a convenience function, if not in native, then in JS.
        Hide
        Filip Maj added a comment -

        Since Camera.getPicture is a PhoneGap API not based on any W3C spec, we can make whatever changes we deem necessary.

        That said, there were plans to deprecate this API and suggest to users to roll with the Capture API, which, as far as I understand, does not take into account any of the issues brought up here.

        Show
        Filip Maj added a comment - Since Camera.getPicture is a PhoneGap API not based on any W3C spec, we can make whatever changes we deem necessary. That said, there were plans to deprecate this API and suggest to users to roll with the Capture API, which, as far as I understand, does not take into account any of the issues brought up here.
        Hide
        Shazron Abdullah added a comment -

        Tentatively assigning to 1.9.0 version

        Show
        Shazron Abdullah added a comment - Tentatively assigning to 1.9.0 version
        Hide
        Shazron Abdullah added a comment -

        Thread here: http://apache.markmail.org/thread/eyqw32hb7yvrlndm

        As stated in the thread, I'll implement the clean() function, which other platforms will just noop.

        Show
        Shazron Abdullah added a comment - Thread here: http://apache.markmail.org/thread/eyqw32hb7yvrlndm As stated in the thread, I'll implement the clean() function, which other platforms will just noop.
        Show
        Shazron Abdullah added a comment - iOS commit - http://git-wip-us.apache.org/repos/asf/incubator-cordova-ios/commit/d8afe9b4 cordova-js commit - http://git-wip-us.apache.org/repos/asf/incubator-cordova-js/commit/5f70310f docs commit - http://git-wip-us.apache.org/repos/asf/incubator-cordova-docs/commit/bd672708

          People

          • Assignee:
            Shazron Abdullah
            Reporter:
            Becky Gibson
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development