Aggregators should not provide .application files for child OA

Bug #1415047 reported by Marcus Tomlinson
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Thomas Strehl
The Savilerow project
Fix Released
High
Kyle Nitzsche

Bug Description

During the confusion of log-in buttons not working in aggregator scopes, a (incorrect) work-around to the problem was to also add the aggregator as a client to the online service (hence you see both FB Photos and Photos under the Facebook account).

Aggregator scopes should not be registering themselves as clients to services, only the scopes directly requiring access should.

Therefore, aggregator scopes need to be updated to remove their .application files so that we no longer see multiple entries in OA settings for one OA client.

description: updated
Changed in savilerow:
assignee: nobody → Kyle Nitzsche (knitzsche)
Changed in canonical-devices-system-image:
importance: Undecided → High
milestone: none → ww07-2015
status: New → Confirmed
Changed in canonical-devices-system-image:
milestone: ww07-2015 → ww09-2015
Changed in savilerow:
status: New → Confirmed
importance: Undecided → High
status: Confirmed → In Progress
Changed in canonical-devices-system-image:
assignee: nobody → John McAleely (john.mcaleely)
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

There is one issue I think:

I removed the application file from photos agg.

Launched it and saw login button for FB as expected. Tapped it and completed log in successfully and was returned to scope.

Refreshed the scope.

Problem:
 * login button still present

Expectation:
* normal logged in results display.

Note: I manually wen to Settings > Accounts > Facebook and saw the FB Photos App. I was not enabled. I enabled it. After doing this, the Photos agg scope shows FB photos.

I don't think the user should be required to navigated to settings to enable the app they just logged into. They should be returned to the agg scope and it should be enabled, so they just see results.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

fixed with com.canonical.scopes.photos_1.29_armhf.click for savilerow

(I assume the above issue I mentioned has nothing to do with the scope.)

Changed in savilerow:
status: In Progress → Fix Committed
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

We cannot have this behavior mentioned in comment #1 get into production, so I am going to pull com.canonical.scopes.photos_1.29_armhf.click

We will need to push a click when this new issue is fixed.

Also, Today agg scope will need the change too.

Changed in savilerow:
status: Fix Committed → Won't Fix
status: Won't Fix → In Progress
Changed in canonical-devices-system-image:
assignee: John McAleely (john.mcaleely) → nobody
Changed in canonical-devices-system-image:
assignee: nobody → Thomas Strehl (strehl-t)
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

Kyle, please ensure that you have the following package versions installed:

unity-plugin-scopes : 0.5.4+15.04.20150206.1~rtm-0ubuntu1
libunity-scopes3 : 0.6.9+15.04.20150126~rtm-0ubuntu1

if not, please upgrade and try again, Photos Scope v1.29 works fine for me.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

here is what I have:
phablet@ubuntu-phablet:~$ apt-cache policy unity-plugin-scopes
unity-plugin-scopes:
  Installed: 0.5.4+15.04.20150126~rtm.is.0.5.4+15.04.20150120~rtm-0ubuntu1

^^^ not the latest

phablet@ubuntu-phablet:~$ apt-cache policy libunity-scopes3
libunity-scopes3:
  Installed: 0.6.9+15.04.20150126~rtm-0ubuntu1

I'll upgrade and report back.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

upgrading unity-plugin-scopes to the stated version did not fix the issue. I'll wipe and reflash and see if that does it.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

after reflash with --wipe, photos v1.29 works as expected regarding FB OA. That is, the scope displays the login button, I tap it, I log in, I return to scope, and fb photos display in photos agg, with no need to manually enable the app in Settings > Accounts

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

I committed photos v1.29 to our clicks branch (the starting point for inclusion in images).

Next up: Today scope needs the same fix.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

today scope now has same fix in clicks branch with com.canonical.scopes.dashboard_1.6_all.click

Changed in savilerow:
status: In Progress → Fix Committed
Thomas Strehl (strehl-t)
Changed in canonical-devices-system-image:
status: Confirmed → Fix Released
Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

mark as fix release since it's probably be.

Changed in savilerow:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.