-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Saving a photo #55
Comments
Yes Please, I'd like this feature too! Love the App |
Yes there is no save option & when trying to use Save on Device to try to save something I just get a blank file with 0 bytes. I thought it was their app so i submitted an issue but they told me it's because of your apps MIME generation its something you must fix within the vector camera app. Don't abandon this software. |
That's not the issue. This Stackoverflow answer provides insight into the root cause of the actual issue. I did a little debugging and this is what I found... The app has a method that tries to index the files to make them accessible to other apps, I think because the images and videos from the app are being stored in private app specific storage, it's conflicting with this method. It would be best if the MediaStore API was used for saving images/videos, it would eliminate the need for apps like SaveTo and SaveOnDevice altogether ref: 1 |
I'm currently working on a PR that should resolve issues 1, 2. The plan is to update The directory structure will look like this: /**
* Directory structure:
*
* // if (Build.VERSION.SDK_INT < Build.VERSION_CODES.Q)
* [app-directory]/
* thumbnails/
* [image_id].jpg
* [video_id].jpg
* metadata/
* [image_id].json
* [video_id].json
* raw/
* [image_id].gz
* [video_id]_video.dat
* [video_id]_audio.pcm
* VectorCamera_images/
* [image_id].png
* VectorCamera_videos/
* [video_id].webm (if exported)
* tmp/
* [video ID of recording in progress]_video.dat
* [video ID of recording in progress]_audio.pcm
*
* // if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q)
* [app-directory]/
* thumbnails/
* [image_id].jpg
* [video_id].jpg
* metadata/
* [image_id].json
* [video_id].json
* raw/
* [image_id].gz
* [video_id]_video.dat
* [video_id]_audio.pcm
* tmp/
* [video ID of recording in progress]_video.dat
* [video ID of recording in progress]_audio.pcm
* [mediaStore]/
* DCIM/
* VectorCamera/
* Images/
* [image_id].png
* Videos/
* [video_id].webm
*
*/ I'm estimating this will take me about 1-2 |
@DeMoN100 @TGHepburn @guglovich Are you all running Android 10 or higher? I just want to confirm that the issue of not being to accessible media files outside of the app is exclusive to non-rooted Android 10+ devices |
Yes that is correct I am on android 12. |
No, I'm on an older android, version 8. I can get to the folder..when I remember where it is ( l know a prompt comes up giving directory but I miss it half the time). I'd just like to choose where they are placed in settings for app |
@lmj0011 thanks for looking into this. Your approach sounds good and I'll be happy to review your PR. |
This issue is kinda asking for 2 seperate things, I plan on handling the first one. Images and Videos will get saved in the |
I have Android 12.1 with Root |
Here's my (work in progress) PR: #56 I've reached a roadblock with trying to save videos into mediaStore, to therefore make them accessible in the public My work on this feature is at an end, since there needs to be more code rewriting done than I'm willing to contribute my spare time to. Hopefully my PR is a good starting point for @dozingcat or another contributor. |
#58 should allow "Save on device" to work. |
@lmj0011 Thanks for your work! The next release will be out shortly and will update to the current API level 33, still using private app storage. After that I'll see if it's possible to use MediaStore based on your PR. |
Please make the option to save photos to the catalog. I understand that now they are stored in the system folder. I would like to choose my own folder.
The text was updated successfully, but these errors were encountered: