diff options
author | Andy Shaw <andy.shaw@qt.io> | 2020-07-17 14:01:04 +0200 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2020-08-07 16:04:02 +0000 |
commit | bb1d888e8e4a38d48f3cfdf7dbc6f14647b12f04 (patch) | |
tree | b22450ad7a363aa93a2457435e79f2ef46cf05d0 /src/plugins/platforms | |
parent | 13d3f361e2381136532d8abee23622a14de24624 (diff) | |
download | qtbase-bb1d888e8e4a38d48f3cfdf7dbc6f14647b12f04.tar.gz |
Android: Add support for travesing directories and accessing files
This enables QDir and QFileInfo/QFile to work with entries found from
a entryList() as it will obtain the permission for these files correctly
when it is encountered.
Due to what seems to be a quirk in the Android API, we cache whenever we
come across a known directory to save time later on for checking if it
is a directory. All uris accessed where we get permissions for are
cached so we get the right URI for that given content url as some are
granted via the Intent.
Fixes: QTBUG-85538
Fixes: QTBUG-83041
Fixes: QTBUG-76886
Change-Id: I685b3c60804812a0e4b85fbdbb4ec5efaa09420c
Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
(cherry picked from commit ac17e62f6512eed6e5c63263fab0bf3c896761aa)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'src/plugins/platforms')
-rw-r--r-- | src/plugins/platforms/android/androidcontentfileengine.cpp | 79 | ||||
-rw-r--r-- | src/plugins/platforms/android/androidcontentfileengine.h | 15 |
2 files changed, 91 insertions, 3 deletions
diff --git a/src/plugins/platforms/android/androidcontentfileengine.cpp b/src/plugins/platforms/android/androidcontentfileengine.cpp index 3e3bdc2592..e552b8fa86 100644 --- a/src/plugins/platforms/android/androidcontentfileengine.cpp +++ b/src/plugins/platforms/android/androidcontentfileengine.cpp @@ -92,13 +92,21 @@ AndroidContentFileEngine::FileFlags AndroidContentFileEngine::fileFlags(FileFlag { FileFlags commonFlags(ReadOwnerPerm|ReadUserPerm|ReadGroupPerm|ReadOtherPerm|ExistsFlag); FileFlags flags; - const bool exists = QJNIObjectPrivate::callStaticMethod<jboolean>( + const bool isDir = QJNIObjectPrivate::callStaticMethod<jboolean>( + "org/qtproject/qt5/android/QtNative", "checkIfDir", + "(Landroid/content/Context;Ljava/lang/String;)Z", QtAndroidPrivate::context(), + QJNIObjectPrivate::fromString(fileName(DefaultName)).object()); + // If it is a directory then we know it exists so there is no reason to explicitly check + const bool exists = isDir ? true : QJNIObjectPrivate::callStaticMethod<jboolean>( "org/qtproject/qt5/android/QtNative", "checkFileExists", "(Landroid/content/Context;Ljava/lang/String;)Z", QtAndroidPrivate::context(), QJNIObjectPrivate::fromString(fileName(DefaultName)).object()); - if (!exists) + if (!exists && !isDir) return flags; - flags = FileType | commonFlags; + if (isDir) + flags = DirectoryType | commonFlags; + else + flags = FileType | commonFlags; return type & flags; } @@ -122,6 +130,16 @@ QString AndroidContentFileEngine::fileName(FileName f) const } } +QAbstractFileEngine::Iterator *AndroidContentFileEngine::beginEntryList(QDir::Filters filters, const QStringList &filterNames) +{ + return new AndroidContentFileEngineIterator(filters, filterNames); +} + +QAbstractFileEngine::Iterator *AndroidContentFileEngine::endEntryList() +{ + return nullptr; +} + AndroidContentFileEngineHandler::AndroidContentFileEngineHandler() = default; AndroidContentFileEngineHandler::~AndroidContentFileEngineHandler() = default; @@ -133,3 +151,58 @@ QAbstractFileEngine* AndroidContentFileEngineHandler::create(const QString &file return new AndroidContentFileEngine(fileName); } + +AndroidContentFileEngineIterator::AndroidContentFileEngineIterator(QDir::Filters filters, + const QStringList &filterNames) + : QAbstractFileEngineIterator(filters, filterNames) +{ +} + +AndroidContentFileEngineIterator::~AndroidContentFileEngineIterator() +{ +} + +QString AndroidContentFileEngineIterator::next() +{ + if (!hasNext()) + return QString(); + ++m_index; + return currentFilePath(); +} + +bool AndroidContentFileEngineIterator::hasNext() const +{ + if (m_index == -1) { + if (path().isEmpty()) + return false; + const bool isDir = QJNIObjectPrivate::callStaticMethod<jboolean>( + "org/qtproject/qt5/android/QtNative", "checkIfDir", + "(Landroid/content/Context;Ljava/lang/String;)Z", + QtAndroidPrivate::context(), + QJNIObjectPrivate::fromString(path()).object()); + if (isDir) { + QJNIObjectPrivate objArray = QJNIObjectPrivate::callStaticObjectMethod("org/qtproject/qt5/android/QtNative", + "listContentsFromTreeUri", + "(Landroid/content/Context;Ljava/lang/String;)[Ljava/lang/String;", + QtAndroidPrivate::context(), + QJNIObjectPrivate::fromString(path()).object()); + if (objArray.isValid()) { + QJNIEnvironmentPrivate env; + const jsize length = env->GetArrayLength(static_cast<jarray>(objArray.object())); + for (int i = 0; i != length; ++i) { + m_entries << QJNIObjectPrivate(env->GetObjectArrayElement( + static_cast<jobjectArray>(objArray.object()), i)).toString(); + } + } + } + m_index = 0; + } + return m_index < m_entries.size(); +} + +QString AndroidContentFileEngineIterator::currentFileName() const +{ + if (m_index <= 0 || m_index > m_entries.size()) + return QString(); + return m_entries.at(m_index - 1); +} diff --git a/src/plugins/platforms/android/androidcontentfileengine.h b/src/plugins/platforms/android/androidcontentfileengine.h index 09e5d77553..31eaf9b0ab 100644 --- a/src/plugins/platforms/android/androidcontentfileengine.h +++ b/src/plugins/platforms/android/androidcontentfileengine.h @@ -50,6 +50,8 @@ public: qint64 size() const override; FileFlags fileFlags(FileFlags type = FileInfoAll) const override; QString fileName(FileName file = DefaultName) const override; + QAbstractFileEngine::Iterator *beginEntryList(QDir::Filters filters, const QStringList &filterNames) override; + QAbstractFileEngine::Iterator *endEntryList() override; private: QString m_file; @@ -63,4 +65,17 @@ public: QAbstractFileEngine *create(const QString &fileName) const override; }; +class AndroidContentFileEngineIterator : public QAbstractFileEngineIterator +{ +public: + AndroidContentFileEngineIterator(QDir::Filters filters, const QStringList &filterNames); + ~AndroidContentFileEngineIterator(); + QString next() override; + bool hasNext() const override; + QString currentFileName() const override; +private: + mutable QStringList m_entries; + mutable int m_index = -1; +}; + #endif // ANDROIDCONTENTFILEENGINE_H |