Skip to content

Conversation

@tobiasKaminsky
Copy link
Member

@tobiasKaminsky tobiasKaminsky commented Jun 17, 2025

  • Tests written, or not not needed

Changed areas

  • Dashboard widget
  • Widget list
  • federated avatars
  • background image in drawer
  • external icons in drawer
  • background image in user info
  • activity list (adapter)
  • notification list (adapter)
  • richdocuments template (adapter)
  • Unified search items
  • backup list

@tobiasKaminsky tobiasKaminsky changed the title initial glide v4 Glide v4 Jun 17, 2025
@alperozturk96 alperozturk96 added performance 🚀 Performance improvement opportunities (non-crash related) technical debt labels Jun 23, 2025
@alperozturk96
Copy link
Collaborator

alperozturk96 commented Jun 23, 2025

  • ksp annotation processor used instead of kapt https://bumptech.github.io/glide/doc/generatedapi.html
  • missing components registered to the NextcloudGlideModule
  • missing animation added
  • wrong glide usages fixed
  • java classes converted to the kotlin
  • CustomTarget used instead of deprecated SimpleTarget
  • missing request listeners added

@alperozturk96
Copy link
Collaborator

Screenshot 2025-06-23 at 14 53 44

@tobiasKaminsky I can't verify new glide dependencies. Tried via following commands:

./gradlew --console=plain --dependency-verification lenient -q --write-verification-metadata sha256,pgp help

./gradlew  --write-verification-metadata pgp,sha256 help

./gradlew --write-verification-metadata pgp,sha256 help --export-keys

./gradlew clean --refresh-dependencies

@tobiasKaminsky

This comment was marked as resolved.

@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 🤖 🍏 Mobile clients team Jun 25, 2025
@tobiasKaminsky tobiasKaminsky moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 🤖 🍏 Mobile clients team Jun 25, 2025
@alperozturk96
Copy link
Collaborator

1
2
3

@tobiasKaminsky Looks good to me, what do you think?

@alperozturk96 alperozturk96 force-pushed the glideV4 branch 3 times, most recently from 9a09bde to 5d06f3e Compare June 27, 2025 05:27
@tobiasKaminsky

This comment was marked as resolved.

@tobiasKaminsky

This comment was marked as resolved.

@alperozturk96
Copy link
Collaborator

alperozturk96 commented Jun 30, 2025

External link not working:

createRequestBuilder: Load failed for https://daily.ltd2.nextcloud.com/apps/external/icons/external.svg
2025-06-30 09:43:25.452  9899-9899  GlideHelper             com.nextcloud.client                 E  createRequestBuilder: Error: Failed to load resource
                                                                                                    There was 1 root cause:
                                                                                                    com.bumptech.glide.load.HttpException(Failed to connect or obtain data, status code: 401)
                                                                                                     call GlideException#logRootCauses(String) for more detail
2025-06-30 09:43:25.452  9899-9899  GlideHelper             com.nextcloud.client                 I  Root cause (1 of 1)
                                                                                                    com.bumptech.glide.load.HttpException: Failed to connect or obtain data, status code: 401
                                                                                                    	at com.bumptech.glide.load.data.HttpUrlFetcher.loadDataWithRedirects(HttpUrlFetcher.java:98)
                                                                                                    	at com.bumptech.glide.load.data.HttpUrlFetcher.loadData(HttpUrlFetcher.java:58)
                                                                                                    	at com.bumptech.glide.load.engine.SourceGenerator.startNextLoad(SourceGenerator.java:95)
                                                                                                    	at com.bumptech.glide.load.engine.SourceGenerator.startNext(SourceGenerator.java:88)
                                                                                                    	at com.bumptech.glide.load.engine.DecodeJob.runGenerators(DecodeJob.java:311)
                                                                                                    	at com.bumptech.glide.load.engine.DecodeJob.runWrapped(DecodeJob.java:280)
                                                                                                    	at com.bumptech.glide.load.engine.DecodeJob.run(DecodeJob.java:235)
                                                                                                    	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
                                                                                                    	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
                                                                                                    	at com.bumptech.glide.load.engine.executor.GlideExecutor$DefaultThreadFactory$1.run(GlideExecutor.java:424)
                                                                                                    	at java.lang.Thread.run(Thread.java:919)
                                                                                                    	at com.bumptech.glide.load.engine.executor.GlideExecutor$DefaultPriorityThreadFactory$1.run(GlideExecutor.java:383)
                                                                                                    Caused by: java.io.FileNotFoundException: https://daily.ltd2.nextcloud.com/apps/external/icons/external.svg
                                                                                                    	at com.android.okhttp.internal.huc.HttpURLConnectionImpl.getInputStream(HttpURLConnectionImpl.java:255)

You can check on our LTD daily instance.

Fixed here is the external.svg

forum

Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
alperozturk96 and others added 2 commits July 7, 2025 11:46
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: tobiasKaminsky <[email protected]>
@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/15017.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link

@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings5148
Errors1111

SpotBugs

CategoryBaseNew
Bad practice6565
Correctness6262
Dodgy code300300
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3535
Performance4848
Security1818
Total538538

@tobiasKaminsky tobiasKaminsky merged commit 1d09343 into master Jul 11, 2025
15 of 21 checks passed
@tobiasKaminsky tobiasKaminsky deleted the glideV4 branch July 11, 2025 06:21
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 🤖 🍏 Mobile clients team Jul 11, 2025
@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 3.33.0 milestone Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance 🚀 Performance improvement opportunities (non-crash related) technical debt

Projects

Development

Successfully merging this pull request may close these issues.

2 participants