8000 chore: draft at using new API to replace java.io.File by farfromrefug · Pull Request #9661 · NativeScript/NativeScript · GitHub
[go: up one dir, main page]
Skip to content

chore: draft at using new API to replace java.io.File #9661

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

chore: draft at using new API to replace java.io.File #9661

wants to merge 5 commits into from

Conversation

farfromrefug
Copy link
Collaborator
@farfromrefug farfromrefug commented Nov 6, 2021 •
edited
Loading

This is a draft PR at introducing new APIs for Android 11.
With Android 11 java.io.File is rendered useless because of permissions.

This PR replaces java.io.File in most places.
It also moves most file system logics to the Java side to improve perfomances.

The work is not finished yet:

  • i implemented a new org.nativescript.widgets.Utils.getFileStats methods to get all stats for a File or a Folder. Still needs to be plugged on the JS side
  • need to implement enumEntities method on the Java side (i have the code).
  • need to implement removeFile/createFile/mkdirs .... For that i have no idea yet.

I also moved the mobile-ui-base android project to gradle 7.0 with latest features. Thus the changes in java files.

Any help on this is welcome. We need this ASAP

@cla-bot cla-bot bot added the cla: yes label Nov 6, 2021
@rigor789 rigor789 added this to the 8.2 milestone Dec 14, 2021
@xpalacincreditoh
Copy link

Hello,

I'm pending the resolution of this change, oh that I can not upload documents.

Thanks

@d3ividloper
Copy link

Hello I came across with the same issue, please let me know when test were fixed in order to apply the fix to uploads documents.
thanks in advance and Merry Christmas!!!

@xpalacincreditoh xpalacincreditoh mentioned this pull request Jan 5, 2022
Open
@xpalacincreditoh
Copy link

I test to apply changes of PR of /core/file-system/file-system-access.android.ts in a patch and have same error.
Is it possible that I don't use ts i lame from js?

@xpalacincreditoh
Copy link

I test to apply changes of PR of /core/file-system/file-system-access.android.js of JS in a patch and have same error.

@farfromrefug farfromrefug mentioned this pull request Mar 4, 2022
Merged
@NathanWalker NathanWalker modified the milestones: 8.2, 9.0 Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@triniwiz triniwiz

Labels
cla: yes
Projects
None yet
Milestone
9.0
Development

Successfully merging this pull request may close these issues.
6 participants
@farfromrefug @xpalacincreditoh @d3ividloper @NathanWalker @rigor789 @triniwiz
0