Merge lp:~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-final-new into lp:ubuntu-filemanager-app

Proposed by Carlos Jose Mazieri
Status: Merged
Approved by: Arto Jalkanen
Approved revision: 424
Merged at revision: 424
Proposed branch: lp:~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-final-new
Merge into: lp:ubuntu-filemanager-app
Prerequisite: lp:~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-13-new
Diff against target: 12 lines (+1/-1)
1 file modified
src/app/qml/ui/FolderListPage.qml (+1/-1)
To merge this branch: bzr merge lp:~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-final-new
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Ubuntu File Manager Developers Pending
Review via email: mp+262526@code.launchpad.net

Commit message

It allows the filemager to browse Samba items by entering any valid url starting with "smb://" or "cifs://".

Description of the change

It replaces https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-final/+merge/253131

It allows the filemager to browse Samba items by entering any valid url starting with "smb://" or "cifs://".

To test: Menu -> GoTo, type smb:// or cifs://

The property "isDir" should be changed in some places to "isBrowsable", the difference is that "isDir" allows remove/copy/cut while "isBrowsable" just means the item is a container and may have sub items.

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app/qml/ui/FolderListPage.qml'
--- src/app/qml/ui/FolderListPage.qml 2015-03-04 10:02:30 +0000
+++ src/app/qml/ui/FolderListPage.qml 2015-06-20 19:32:16 +0000
@@ -888,7 +888,7 @@
888 }888 }
889889
890 function itemClicked(model) {890 function itemClicked(model) {
891 if (model.isDir) {891 if (model.isBrowsable) {
892 if (model.isReadable && model.isExecutable) {892 if (model.isReadable && model.isExecutable) {
893 console.log("Changing to dir", model.filePath)893 console.log("Changing to dir", model.filePath)
894 goTo(model.filePath)894 goTo(model.filePath)

Subscribers

People subscribed via source and target branches