Reset Registration with school servers - short term solution

[kimquirk]

Laptops should automatically register and re-register with school servers when they detect that they are connected to a new one. In today's builds (8.2), you have to push a button to register, and there is no way to unregister or reregister with a new school.

This is the feature request that matches up with the short term fix for 8.2 to allow resetting of the registration through the control panel.

[gregorio]

Please make a CLI option for this as discussed in the recent release meeting.

[kimquirk]

To clarify, this is the short term fix request. I just need the one CLI command to remove the schoolserver items from the config file. This will not deal with issues on the school server.

[erikos]

cli command to reset the school server

[marco]

I think the options handling could be cleanup up a lot, but this is consistent with the code we have there right now and it's no good time for refactoring. We should open a separate ticket about it.

I don't think the patch really works. Command line is in a separate process. I guess you should save the profile and require a restart.

[erikos]

added restart and saving of the profile

[erikos]

pushed to git

[erikos]

|TestCase|

connect to a schoolserver using the 'Register' option in the xo-palette on the home-view. open a terminal activity and use the command line interface of the control panel to reset the registration 'sugar-control-panel -c registration'. After restarting sugar try to connect to the schoolserver again.

[mchua]

assigning to frances to test and verify in release candidate - see above testcase by erikos.

[mchua]

This doesn't seem to be fixed yet. On a freshly installed (copy-nand) 765 (gg-765-2, to be exact) running the test case erikos put up does not pass, as the 'Register' option doesn't reappear in the XO-palette in Home view after running the '-c registration' command. (Am I missing something?)

I also tried (separately) typing the sugar-control-panel command in Terminal as root, and rebooting the entire machine (instead of just restarting Sugar), to no avail. Handing back to erikos.

[gregorio]

Hi Erikos,

Is this fixed?

Why can't we verify it?

Thanks,

Greg S

[kimquirk]

Doesn't work for me in 8.2-767.

[erikos]

make the default value for the backup url an empty string

[erikos]

The patch above for sugar-toolkit (which makes the backup-url an empty string by default) and the change (pro.backup1 = ) below in sugar should fix the issue. Tested on the XO, without an actual schoolserver though.

def clear_registration():
    """Clear the registration with the schoolserver                                                                                         
    """
    pro = profile.get_profile()
    pro.backup1 = None
    pro.save()
    return 1

So to fix this we would need a new sugar and sugar-toolkit package :/

[erikos]

This is the clearer patch for sugar.

diff --git a/src/controlpanel/model/network.py b/src/controlpanel/model/network.py
index cbe0473..b70c952 100644
--- a/src/controlpanel/model/network.py
+++ b/src/controlpanel/model/network.py
@@ -87,7 +87,7 @@ def clear_registration():
     """Clear the registration with the schoolserver
     """
     pro = profile.get_profile()
-    pro.backup1 = None
+    pro.backup1 = ''
     pro.save()
     return 1

[marco]

Simon, can you explain why the current code does not work.

[erikos]

We set it to pro.backup1 to None when we reset the registration, but this is never written to the profile.

http://dev.laptop.org/git?p=sugar-toolkit;a=blob;f=src/sugar/profile.py;h=1b08202c8ecedd63694d73412dfdd7778b81a6d5;hb=9e852bbbbd95baba541390faba3d08cafd09df95#l105

[erikos]

After discussion with marco, we decided that removing the option might be the better solution. This actually would only need a change in sugar-toolkit as well.

diff --git a/src/sugar/profile.py b/src/sugar/profile.py
index 1b08202..6c5dcd2 100644
--- a/src/sugar/profile.py
+++ b/src/sugar/profile.py
@@ -102,8 +102,13 @@ class Profile(object):
             _set_key(cp, 'Buddy', 'NickName', self.nick_name.encode('utf8'))
         if self.color:
             _set_key(cp, 'Buddy', 'Color', self.color.to_string())
-        if self.backup1:
+
+        if self.backup1 is None:
+            if cp.has_option('Server', 'Backup1'):
+                cp.remove_section('Server')
+        else:
             _set_key(cp, 'Server', 'Backup1', self.backup1)
+
         if self.jabber_server is not None:
             _set_key(cp, 'Jabber', 'Server', self.jabber_server)
 

[erikos]

joyride-2513 has the fix. Can someone with a schoolserver please test?

[mchua]

tested with joyride-2513 - verified w/ XS at 1cc.