Merge lp:~carlos-mazieri/ubuntu-filemanager-app/samba-actions-02 into lp:ubuntu-filemanager-app
Proposed by
Carlos Jose Mazieri
Status: | Merged |
---|---|
Approved by: | Arto Jalkanen |
Approved revision: | 432 |
Merged at revision: | 437 |
Proposed branch: | lp:~carlos-mazieri/ubuntu-filemanager-app/samba-actions-02 |
Merge into: | lp:ubuntu-filemanager-app |
Prerequisite: | lp:~carlos-mazieri/ubuntu-filemanager-app/samba-actions-01 |
Diff against target: |
413 lines (+118/-52) 7 files modified
src/plugin/folderlistmodel/diriteminfo.cpp (+3/-3) src/plugin/folderlistmodel/dirmodel.cpp (+9/-9) src/plugin/folderlistmodel/location.cpp (+24/-1) src/plugin/folderlistmodel/location.h (+41/-0) src/plugin/folderlistmodel/locationsfactory.cpp (+21/-15) src/plugin/folderlistmodel/locationsfactory.h (+12/-16) src/plugin/test_folderlistmodel/regression/tst_folderlistmodel.cpp (+8/-8) |
To merge this branch: | bzr merge lp:~carlos-mazieri/ubuntu-filemanager-app/samba-actions-02 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Arto Jalkanen | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email: mp+265193@code.launchpad.net |
Commit message
Enumerator Locations moved from class LocationsFactory to class Location
Created Location:
Created Location:
Description of the change
Location class is improved:
* Some methods from FileSystemAction will migrate to Location class
* Added methods to identify if a Location object is Local/Remote
To post a comment you must log in.
PASSED: Continuous integration, rev:432 91.189. 93.70:8080/ job/ubuntu- filemanager- app-ci/ 508/ 91.189. 93.70:8080/ job/generic- mediumtests- utopic/ 3092 91.189. 93.70:8080/ job/generic- mediumtests- utopic/ 3092/artifact/ work/output/ *zip*/output. zip 91.189. 93.70:8080/ job/ubuntu- filemanager- app-utopic- amd64-ci/ 224 91.189. 93.70:8080/ job/ubuntu- filemanager- app-vivid- amd64-ci/ 130
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/ubuntu- filemanager- app-ci/ 508/rebuild
http://