Filemanager sending out file:/// URLs
Bug #1366217 reported by
Ted Gould
This bug affects 4 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu File Manager App |
Fix Released
|
High
|
Arto Jalkanen |
Bug Description
It seems the file manager is sending file:/// URLs to the URL Dispatcher. Those are unlikely to work and generally a bad idea. For viewing files the file manager should be using the content hub.
https:/
Related branches
lp://staging/~ajalkane/ubuntu-filemanager-app/open-files-using-content-hub
- Carlos Jose Mazieri: Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
-
Diff: 186 lines (+148/-6)4 files modifiedsrc/app/qml/content-hub/FileOpener.qml (+71/-0)
src/app/qml/content-hub/contenttyperesolver.js (+73/-0)
src/app/qml/filemanager.qml (+4/-0)
src/app/qml/ui/FolderListPage.qml (+0/-6)
Changed in ubuntu-filemanager-app: | |
assignee: | nobody → Carlos Jose Mazieri (carlos-mazieri) |
Changed in ubuntu-filemanager-app: | |
status: | New → Triaged |
Changed in ubuntu-filemanager-app: | |
assignee: | Carlos Jose Mazieri (carlos-mazieri) → Arto Jalkanen (ajalkane) |
status: | Triaged → In Progress |
Changed in ubuntu-filemanager-app: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
File manager uses QDestktpService s::openUrl( ) to open files: http:// bazaar. launchpad. net/~ubuntu- filemanager- dev/ubuntu- filemanager- app/trunk/ view/head: /src/plugin/ folderlistmodel /dirmodel. cpp#L1314
For files it uses file:/// prefix and it is necessary.
It looks like QDestktpService s::openUrl( ) is translated to the URL Dispatcher.
Is it what you are talking about?