Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/set-bzr-revno into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
Review: Approve

Nice, Thanks.
-- 
https://code.launchpad.net/~popey/ubuntu-filemanager-app/set-bzr-revno/+merge/274957
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into lp:ubuntu-filemanager-app

2015-10-20 Thread Alan Pope 
Alan Pope  has proposed merging 
lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into 
lp:ubuntu-filemanager-app.

Commit message:
Found a few more places in the UI where the OSK uses predictive text. Disabled 
these.

Requested reviews:
  Ubuntu File Manager Developers (ubuntu-filemanager-dev)

For more details, see:
https://code.launchpad.net/~popey/ubuntu-filemanager-app/suppress-predictive-text/+merge/274995
-- 
Your team Ubuntu File Manager Developers is requested to review the proposed 
merge of lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into 
lp:ubuntu-filemanager-app.
=== modified file 'src/app/qml/ui/GoToDialog.qml'
--- src/app/qml/ui/GoToDialog.qml	2014-11-30 10:26:19 +
+++ src/app/qml/ui/GoToDialog.qml	2015-10-20 10:05:20 +
@@ -36,7 +36,7 @@
 id: locationField
 objectName: "inputField"
 
-inputMethodHints: Qt.ImhNoAutoUppercase
+inputMethodHints: Qt.ImhNoAutoUppercase | Qt.ImhNoPredictiveText
 
 property bool valid: pathExists(text)
 

=== modified file 'src/app/qml/ui/PlacesPage.qml'
--- src/app/qml/ui/PlacesPage.qml	2015-01-06 19:56:17 +
+++ src/app/qml/ui/PlacesPage.qml	2015-10-20 10:05:20 +
@@ -54,7 +54,7 @@
 margins: units.gu(1)
 }
 
-inputMethodHints: Qt.ImhNoAutoUppercase
+inputMethodHints: Qt.ImhNoAutoUppercase | Qt.ImhNoPredictiveText
 
 property bool valid: pathExists(text)
 

=== modified file 'src/app/qml/ui/ViewPopover.qml'
--- src/app/qml/ui/ViewPopover.qml	2014-09-20 10:49:51 +
+++ src/app/qml/ui/ViewPopover.qml	2015-10-20 10:05:20 +
@@ -140,7 +140,7 @@
 margins: units.gu(1)
 }
 
-inputMethodHints: Qt.ImhNoAutoUppercase
+inputMethodHints: Qt.ImhNoAutoUppercase | Qt.ImhNoPredictiveText
 
 text: pageModel.nameFilters
 

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into lp:ubuntu-filemanager-app

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:482
http://91.189.93.70:8080/job/ubuntu-filemanager-app-ci/562/
Executed test runs:
SUCCESS: 
http://91.189.93.70:8080/job/ubuntu-filemanager-app-vivid-amd64-ci/184

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-filemanager-app-ci/562/rebuild

-- 
https://code.launchpad.net/~popey/ubuntu-filemanager-app/suppress-predictive-text/+merge/274995
Your team Ubuntu File Manager Developers is requested to review the proposed 
merge of lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
Review: Approve

OK, Thanks.
-- 
https://code.launchpad.net/~popey/ubuntu-filemanager-app/suppress-predictive-text/+merge/274995
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Stefano Verzegnassi
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init into 
lp:ubuntu-docviewer-app/reboot has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init/+merge/274806
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread noreply
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init into 
lp:ubuntu-docviewer-app/reboot has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init/+merge/274806
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Stefano Verzegnassi
Review: Approve

Ok, now it works. Also, I gave a run to the Autopilot tests on my PC and 
everything was okay.
-- 
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-app-init/+merge/274806
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread noreply
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into 
lp:ubuntu-docviewer-app/reboot has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12/+merge/274051
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration


-- 
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12/+merge/274051
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
Review: Approve


-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275084
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275084
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Bug 1472677] Re: Network shares are inaccessable by the application

2015-10-20 Thread Carlos Jose Mazieri
It has been released.

** Changed in: ubuntu-filemanager-app
   Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
File Manager Developers, which is subscribed to Ubuntu File Manager App.
Matching subscriptions: File Manager App Bugmail
https://bugs.launchpad.net/bugs/1472677

Title:
  Network shares are inaccessable by the application

Status in Ubuntu File Manager App:
  Fix Released

Bug description:
  In it's current form, the File Manager application is unable to see or
  access AFP network shares. I don't have any SAMBA shares on my home
  network, so I am unable to confirm whether or not the same would also
  be true for this type of network share. I suspect it would as there
  are no options in any of menus or settings to connect to a network
  share. "Places" within the app would seem like a logical place to find
  the "Network" as a place.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-filemanager-app/+bug/1472677/+subscriptions

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
Review: Approve


-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01/+merge/27
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Autolanding.
More details in the following jenkins job:
http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-autolanding/31/
Executed test runs:
FAILURE: 
http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-vivid-amd64-autolanding/31/console

-- 
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12/+merge/274051
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into 
lp:ubuntu-docviewer-app/reboot has been updated.

Status: Approved => Needs review

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12/+merge/274051
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Roman Shchekin
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into 
lp:ubuntu-docviewer-app/reboot has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12/+merge/274051
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
Review: Approve

Small nag: Commit message should be "Fixed" not "ixed". 
-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/273337
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration


-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275084
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread noreply
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into 
lp:ubuntu-filemanager-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275084
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01 into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01/+merge/27
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02 into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02/+merge/273334
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
Review: Approve


-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02/+merge/273334
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03 into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03/+merge/273335
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03 into lp:ubuntu-filemanager-app

2015-10-20 Thread Arto Jalkanen
Review: Approve


-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03/+merge/273335
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/set-bzr-revno into lp:ubuntu-filemanager-app

2015-10-20 Thread noreply
The proposal to merge lp:~popey/ubuntu-filemanager-app/set-bzr-revno into 
lp:ubuntu-filemanager-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~popey/ubuntu-filemanager-app/set-bzr-revno/+merge/274957
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:477
http://91.189.93.70:8080/job/ubuntu-filemanager-app-ci/563/
Executed test runs:
SUCCESS: 
http://91.189.93.70:8080/job/ubuntu-filemanager-app-vivid-amd64-ci/185

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-filemanager-app-ci/563/rebuild

-- 
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275084
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03 into lp:ubuntu-filemanager-app

2015-10-20 Thread noreply
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03 into 
lp:ubuntu-filemanager-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03/+merge/273335
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
The proposal to merge lp:~popey/ubuntu-filemanager-app/suppress-predictive-text 
into lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~popey/ubuntu-filemanager-app/suppress-predictive-text/+merge/274995
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Bug 1505594] Re: Wrong time of accessed files

2015-10-20 Thread Carlos Jose Mazieri
There was related bug that was fixed some time ago,  all items used show
the time of the current folder.

I think that it is OK now,  can you check it again?

-- 
You received this bug notification because you are a member of Ubuntu
File Manager Developers, which is subscribed to Ubuntu File Manager App.
Matching subscriptions: File Manager App Bugmail
https://bugs.launchpad.net/bugs/1505594

Title:
  Wrong time of accessed files

Status in Ubuntu File Manager App:
  New

Bug description:
  Hello,
  In example I'm looking at Properties of one of my files (video taken by 
Camera app) and I see:
  Accessed: 23.09.2015 16:03
  Modified: 12.10.2015 13:55

  But file was created at 26.09.2015 14:16

  I think maybe Properties option of the file in real is showing
  Properties of directory that contain that file (because I've checked
  all files in folder com.ubuntu.camera - all have the same Accessed and
  Modified date and time).

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-filemanager-app/+bug/1505594/+subscriptions

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into lp:ubuntu-docviewer-app/reboot

2015-10-20 Thread Roman Shchekin
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12 into 
lp:ubuntu-docviewer-app/reboot has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-uitk12/+merge/274051
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Superseded

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/273337
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01 into lp:ubuntu-filemanager-app

2015-10-20 Thread noreply
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01 into 
lp:ubuntu-filemanager-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-01/+merge/27
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
Carlos Jose Mazieri has proposed merging 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into 
lp:ubuntu-filemanager-app with 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-03 as a 
prerequisite.

Commit message:
ixed SmbUtil::changePermissions()
improved SmbUtil::checkValidShareName()

Requested reviews:
  Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot): 
continuous-integration
  Arto Jalkanen (ajalkane)

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275083

Fixed SmbUtil::changePermissions()
improved SmbUtil::checkValidShareName()
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.
=== modified file 'src/plugin/folderlistmodel/smb/qsambaclient/src/smbutil.cpp'
--- src/plugin/folderlistmodel/smb/qsambaclient/src/smbutil.cpp	2015-10-20 18:31:25 +
+++ src/plugin/folderlistmodel/smb/qsambaclient/src/smbutil.cpp	2015-10-20 18:31:25 +
@@ -342,7 +342,7 @@
 Smb::FileHandler fd = 0;
 ::memset(st, 0, sizeof(struct stat));
 if ((fd=openDir(context, smb_path)) )
-{
+{
 ret = guessDirType(context,fd);
 closeHandle(context, fd);
 if (ret == StatDir)
@@ -356,7 +356,7 @@
 }
 }
 else if (errno != EACCES && errno != ECONNREFUSED) // perhaps is a file
-{
+{
 errno = 0;
 ret = getStat(context, smb_path,st);
 }
@@ -636,6 +636,10 @@
 {
   return false;
 }
+if (::strcmp(shareName, "ADMIN$") == 0)
+{
+  return false;
+}
 
 return true;
 }
@@ -727,7 +731,7 @@
 
 bool SmbUtil::changePermissions(Smb::Context context, const QString& smb_path, mode_t mode)
 {
-int ret = ::smbc_getFunctionChmod(context)(context, smb_path.toLocal8Bit().constBegin(), mode);
+int ret = ::smbc_getFunctionChmod(context)(context, smb_path.toLocal8Bit().constData(), mode);
 if (ret < 0)
 {
 SHOW_ERRNO(smb_path);

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02 into lp:ubuntu-filemanager-app

2015-10-20 Thread noreply
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02 into 
lp:ubuntu-filemanager-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-02/+merge/273334
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/set-bzr-revno into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
The proposal to merge lp:~popey/ubuntu-filemanager-app/set-bzr-revno into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~popey/ubuntu-filemanager-app/set-bzr-revno/+merge/274957
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/suppress-predictive-text into lp:ubuntu-filemanager-app

2015-10-20 Thread noreply
The proposal to merge lp:~popey/ubuntu-filemanager-app/suppress-predictive-text 
into lp:ubuntu-filemanager-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~popey/ubuntu-filemanager-app/suppress-predictive-text/+merge/274995
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into lp:ubuntu-filemanager-app

2015-10-20 Thread Carlos Jose Mazieri
The proposal to merge 
lp:~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04 into 
lp:ubuntu-filemanager-app has been updated.

Status: Needs review => Superseded

For more details, see:
https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-improvements-04/+merge/275083
-- 
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Bug 1505594] Re: Wrong time of accessed files

2015-10-20 Thread Krzysztof Tataradziński
Okey, I can confirm that is fixed. I just looked at old files created
when that bug was occurring. I'm sorry for wasting your time guys! :)

** Changed in: ubuntu-filemanager-app
   Status: New => Fix Released

** Changed in: ubuntu-filemanager-app
   Status: Fix Released => Invalid

-- 
You received this bug notification because you are a member of Ubuntu
File Manager Developers, which is subscribed to Ubuntu File Manager App.
Matching subscriptions: File Manager App Bugmail
https://bugs.launchpad.net/bugs/1505594

Title:
  Wrong time of accessed files

Status in Ubuntu File Manager App:
  Invalid

Bug description:
  Hello,
  In example I'm looking at Properties of one of my files (video taken by 
Camera app) and I see:
  Accessed: 23.09.2015 16:03
  Modified: 12.10.2015 13:55

  But file was created at 26.09.2015 14:16

  I think maybe Properties option of the file in real is showing
  Properties of directory that contain that file (because I've checked
  all files in folder com.ubuntu.camera - all have the same Accessed and
  Modified date and time).

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-filemanager-app/+bug/1505594/+subscriptions

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~vthompson/music-app/music-uc1.3 into lp:music-app

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:901
http://91.189.93.70:8080/job/music-app-ci/1390/
Executed test runs:
SUCCESS: http://91.189.93.70:8080/job/music-app-vivid-amd64-ci/242

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/music-app-ci/1390/rebuild

-- 
https://code.launchpad.net/~vthompson/music-app/music-uc1.3/+merge/267889
Your team Music App Developers is subscribed to branch lp:music-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/refactor-use-sdk-listitems into lp:music-app

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:882
http://91.189.93.70:8080/job/music-app-ci/1391/
Executed test runs:
SUCCESS: http://91.189.93.70:8080/job/music-app-vivid-amd64-ci/243

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/music-app-ci/1391/rebuild

-- 
https://code.launchpad.net/~ahayzen/music-app/refactor-use-sdk-listitems/+merge/274829
Your team Music App Developers is subscribed to branch lp:music-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~vthompson/ubuntu-weather-app/fix-var-name-clash into lp:ubuntu-weather-app/reboot

2015-10-20 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:146
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-ci/283/
Executed test runs:
SUCCESS: 
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-vivid-amd64-ci/283

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-ci/283/rebuild

-- 
https://code.launchpad.net/~vthompson/ubuntu-weather-app/fix-var-name-clash/+merge/275118
Your team Ubuntu Weather Developers is requested to review the proposed merge 
of lp:~vthompson/ubuntu-weather-app/fix-var-name-clash into 
lp:ubuntu-weather-app/reboot.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~vthompson/ubuntu-weather-app/fix-var-name-clash into lp:ubuntu-weather-app/reboot

2015-10-20 Thread Victor Thompson
Victor Thompson has proposed merging 
lp:~vthompson/ubuntu-weather-app/fix-var-name-clash into 
lp:ubuntu-weather-app/reboot.

Commit message:
* Fix variable name clash that caused the hourly forecast to use the wrong 
offset.

Requested reviews:
  Ubuntu Weather Developers (ubuntu-weather-dev)

For more details, see:
https://code.launchpad.net/~vthompson/ubuntu-weather-app/fix-var-name-clash/+merge/275118

* Fix variable name clash that caused the hourly forecast to use the wrong 
offset.

The previous MP caused a regression in the hourly forecast times, such that 
they were offset by the current location's timezone offset.
-- 
Your team Ubuntu Weather Developers is requested to review the proposed merge 
of lp:~vthompson/ubuntu-weather-app/fix-var-name-clash into 
lp:ubuntu-weather-app/reboot.
=== modified file 'app/data/WeatherApi.js'
--- app/data/WeatherApi.js	2015-10-20 01:37:34 +
+++ app/data/WeatherApi.js	2015-10-21 02:20:20 +
@@ -553,9 +553,9 @@
 }
 tmpResult[day] = _buildDayFormat(date, dayData, nowMs);
 var timezoneOffset = new Date().getTimezoneOffset();
-var offset = (location.timezone && location.timezone.dstOffset !== undefined) ? (location.timezone.dstOffset*60 + timezoneOffset)*60*1000: 0
-var sunrise = new Date(sunRiseSet.rise*1000 + offset);
-var sunset = new Date(sunRiseSet.set*1000 + offset);
+var timesOffset = (location.timezone && location.timezone.dstOffset !== undefined) ? (location.timezone.dstOffset*60 + timezoneOffset)*60*1000: 0
+var sunrise = new Date(sunRiseSet.rise*1000 + timesOffset);
+var sunset = new Date(sunRiseSet.set*1000 + timesOffset);
 var options = { timeZone: location.timezone.timeZoneId, timeZoneName: 'long' };
 tmpResult[day].sunrise = sunrise.toLocaleTimeString(Qt.locale().name, options);
 tmpResult[day].sunset = sunset.toLocaleTimeString(Qt.locale().name, options);

=== modified file 'debian/changelog'
--- debian/changelog	2015-10-18 22:57:34 +
+++ debian/changelog	2015-10-21 02:20:20 +
@@ -31,6 +31,7 @@
   * Show the bottom loading animation (LoadingIndicator) on the LocationsPage. (LP: #1490040)
   * Add FakeHeader component to ease the bottom edge transition.
   * Use the timezone of the Location to display sunrise/sunset times.
+  * Fix variable name clash that caused unintented consequences.
 
   [ Andrew Hayzen ]
   * Add mocked locations for autopilot and add a test using the data

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp