[Sugar-devel] [PATCH v8] Downgrading activities not allowed. (SL #2164)

[Aleksey Lim]

On Mon, Oct 25, 2010 at 07:31:27PM +0530, shanjit at seeta.in wrote:
> From: Shanjit Singh Jajmann <shanjit at seeta.in>, Anubhav Aggarwal <anubhav at seeta.in>
> 
> Activity can be downgraded on the availability of an older .xo version of an
> activity. An alert pops up when trying to install an older .xo file of an
> activity, which asks the user to make a selection on whether to move to an
> older activity version or not.
> 
> Co-authored-by: Shanjit Singh Jajmann <shanjit at seeta.in>
> Co-authored-by: Anubhav Aggarwal <anubhav at seeta.in>

Thanks, pushed
(with minor renaming tweak,
http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/5490d41ec310543b1819165f5209020e15411863)

> ---
> 
> v1 -> v2. Inline function used, signal emission condition revised and global
> variables removed. Recommendations by James Cameron and Aleksey Lim added.
> 
> v2 -> v3. Used misc.resume.
> 
> v3 -> v4. Changes in the copyright of the new file.
> 
> v4 -> v5. Alert shown in the same window as the journal.
> 
> v5 -> v6. Static variable removed, name of the functions changed.
> Recommendations by James Cameron and Aleksey Lim added.
> 
> v6 -> v7. Logic for the alert pop up made simpler.
> Recommendations by Aleksay Lim added.
> 
> v7 -> v8. Missing file added.
> ---
>  src/jarabe/journal/journalactivity.py |    5 ++-
>  src/jarabe/journal/journalwindow.py   |   34 +++++++++++++++++++++++
>  src/jarabe/journal/misc.py            |   47 ++++++++++++++++++++++++++------
>  src/jarabe/model/bundleregistry.py    |    7 +++-
>  4 files changed, 80 insertions(+), 13 deletions(-)
>  create mode 100644 src/jarabe/journal/journalwindow.py
> 
> diff --git a/src/jarabe/journal/journalactivity.py b/src/jarabe/journal/journalactivity.py
> index 44cc018..beb0962 100644
> --- a/src/jarabe/journal/journalactivity.py
> +++ b/src/jarabe/journal/journalactivity.py
> @@ -44,6 +44,7 @@ from jarabe.journal.journalentrybundle import JournalEntryBundle
>  from jarabe.journal.objectchooser import ObjectChooser
>  from jarabe.journal.modalalert import ModalAlert
>  from jarabe.journal import model
> +from jarabe.journal.journalwindow import JournalWindow
>  
>  J_DBUS_SERVICE = 'org.laptop.Journal'
>  J_DBUS_INTERFACE = 'org.laptop.Journal'
> @@ -102,10 +103,10 @@ class JournalActivityDBusService(dbus.service.Object):
>      def ObjectChooserCancelled(self, chooser_id):
>          pass
>  
> -class JournalActivity(Window):
> +class JournalActivity(JournalWindow):
>      def __init__(self):
>          logging.debug("STARTUP: Loading the journal")
> -        Window.__init__(self)
> +        JournalWindow.__init__(self)
>  
>          self.set_title(_('Journal'))
>  
> diff --git a/src/jarabe/journal/journalwindow.py b/src/jarabe/journal/journalwindow.py
> new file mode 100644
> index 0000000..3c718c2
> --- /dev/null
> +++ b/src/jarabe/journal/journalwindow.py
> @@ -0,0 +1,34 @@
> +#Copyright (C) 2010 Software for Education, Entertainment and Training
> +#Activities
> +#
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; either version 2 of the License, or
> +# (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
> +
> +import gtk
> +from sugar.graphics.window import Window
> +
> +_journal_window = None
> +
> +
> +class JournalWindow(Window):
> +
> +    def __init__(self):
> +
> +        global _journal_window
> +        Window.__init__(self)
> +        _journal_window = self
> +
> +
> +def get_journal_window():
> +    return _journal_window
> diff --git a/src/jarabe/journal/misc.py b/src/jarabe/journal/misc.py
> index 32a2847..122feca 100644
> --- a/src/jarabe/journal/misc.py
> +++ b/src/jarabe/journal/misc.py
> @@ -27,8 +27,10 @@ from sugar.activity import activityfactory
>  from sugar.activity.activityhandle import ActivityHandle
>  from sugar.graphics.icon import get_icon_file_name
>  from sugar.graphics.xocolor import XoColor
> +from sugar.graphics.alert import ConfirmationAlert
>  from sugar import mime
>  from sugar.bundle.activitybundle import ActivityBundle
> +from sugar.bundle.bundle import AlreadyInstalledException
>  from sugar.bundle.contentbundle import ContentBundle
>  from sugar import util
>  
> @@ -36,6 +38,7 @@ from jarabe.view import launcher
>  from jarabe.model import bundleregistry, shell
>  from jarabe.journal.journalentrybundle import JournalEntryBundle
>  from jarabe.journal import model
> +from jarabe.journal import journalwindow
>  
>  def _get_icon_for_mime(mime_type):
>      generic_types = mime.get_all_generic_types()
> @@ -159,19 +162,16 @@ def resume(metadata, bundle_id=None):
>          bundle = ActivityBundle(file_path)
>          if not registry.is_installed(bundle):
>              logging.debug('Installing activity bundle')
> -            registry.install(bundle)
> +            try:
> +                registry.install(bundle)
> +            except AlreadyInstalledException:
> +                _downgrade_option_alert(bundle)
> +                return
>          else:
>              logging.debug('Upgrading activity bundle')
>              registry.upgrade(bundle)
>  
> -        logging.debug('activityfactory.creating bundle with id %r',
> -                        bundle.get_bundle_id())
> -        installed_bundle = registry.get_bundle(bundle.get_bundle_id())
> -        if installed_bundle:
> -            launch(installed_bundle)
> -        else:
> -            logging.error('Bundle %r is not installed.',
> -                          bundle.get_bundle_id())
> +        _install_bundle(bundle)
>  
>      elif is_content_bundle(metadata) and bundle_id is None:
>  
> @@ -215,6 +215,17 @@ def resume(metadata, bundle_id=None):
>          launch(bundle, activity_id=activity_id, object_id=object_id,
>                  color=get_icon_color(metadata))
>  
> +def _install_bundle(bundle):
> +    registry = bundleregistry.get_registry()
> +    logging.debug('activityfactory.creating bundle with id %r',
> +                       bundle.get_bundle_id())
> +    installed_bundle = registry.get_bundle(bundle.get_bundle_id())
> +    if installed_bundle:
> +        launch(installed_bundle)
> +    else:
> +        logging.error('Bundle %r is not installed.',
> +                    bundle.get_bundle_id())
> +
>  def launch(bundle, activity_id=None, object_id=None, uri=None, color=None,
>             invited=False):
>      if activity_id is None or not activity_id:
> @@ -239,6 +250,24 @@ def launch(bundle, activity_id=None, object_id=None, uri=None, color=None,
>              object_id=object_id, uri=uri, invited=invited)
>      activityfactory.create(bundle, activity_handle)
>  
> +def _downgrade_option_alert(bundle):
> +    alert = ConfirmationAlert()
> +    alert.props.title = _('Older Version Of %s Activity') % (bundle.get_name())
> +    alert.props.msg = _('Do you want to downgrade to version %s\
> +    ') % (bundle.get_activity_version())
> +    alert.connect('response', _downgrade_alert_response_cb, bundle)
> +    journalwindow.get_journal_window().add_alert(alert)
> +    alert.show()
> +
> +def _downgrade_alert_response_cb(alert, response_id, bundle):
> +    if response_id is gtk.RESPONSE_OK:
> +        journalwindow.get_journal_window().remove_alert(alert)
> +        registry = bundleregistry.get_registry()
> +        registry.install(bundle, force_downgrade=True)
> +        _install_bundle(bundle)
> +    elif response_id is gtk.RESPONSE_CANCEL:
> +        journalwindow.get_journal_window().remove_alert(alert)
> +
>  def is_activity_bundle(metadata):
>      mime_type = metadata.get('mime_type', '')
>      return mime_type == ActivityBundle.MIME_TYPE or \
> diff --git a/src/jarabe/model/bundleregistry.py b/src/jarabe/model/bundleregistry.py
> index 699e339..4de7e3f 100644
> --- a/src/jarabe/model/bundleregistry.py
> +++ b/src/jarabe/model/bundleregistry.py
> @@ -373,14 +373,17 @@ class BundleRegistry(gobject.GObject):
>                  return True
>          return False
>  
> -    def install(self, bundle, uid=None):
> +    def install(self, bundle, uid=None, force_downgrade=False):
>          activities_path = env.get_user_activities_path()
>  
>          for installed_bundle in self._bundles:
>              if bundle.get_bundle_id() == installed_bundle.get_bundle_id() and \
>                      bundle.get_activity_version() <= \
>                          installed_bundle.get_activity_version():
> -                raise AlreadyInstalledException
> +                if not force_downgrade:
> +                    raise AlreadyInstalledException
> +                else:
> +                    self.uninstall(installed_bundle, force=True)
>              elif bundle.get_bundle_id() == installed_bundle.get_bundle_id():
>                  self.uninstall(installed_bundle, force=True)
>  
> -- 
> 1.7.2.2
> 

-- 
Aleksey