Changeset 5968 for main/waeup.sirp/trunk/src/waeup/sirp/interfaces.py

Timestamp:

23 Apr 2011, 01:46:57 (14 years ago)

Author:

uli

Message:

Replace themes source by themes vocabulary. It looks like this is much easier to handle in the long run.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/interfaces.py (modified) (3 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/interfaces.py

@@ -14 +14 @@
 from waeup.sirp.permissions import RoleSource
 
-class IThemesProvider(Interface):
-    """A component that delivers all themes defined.
-
-    We normally do not deal with browser-related stuff in `waeup.sirp`
-    but leave that to the `browser` subpackage. With themes (aka
-    skins) we cannot follow that path because we need a theme as an
-    attribute of each `University` instance.
-
-    Therefore we must tell the browser subpackage how we expect themes
-    to be announced code-wise.
-
-    What we tell here is that we might ask for a utility providing
-    IThemeProvider and that this component then should be callable to
-    give us a list of themes and their respective internal names.
-    """
-    def __call__():
-        """Get a list of all themes available.
-
-        Returns a list of ``(<NAME>, <THEME>)`` tuples where
-        ``<NAME>`` is the internal name under which a theme was
-        registered and ``<THEME>`` is the real theme.
-
-        Each <THEME> must have a `description` attribute.
-        """
-
-class ThemesSource(BasicSourceFactory):
-    """A source for themes.
-    """
-    def getValues(self):
-        """Get names of themes.
-        """
-        themes_provider = getUtility(IThemesProvider, name="default")
-        return [x for x, y in themes_provider()]
-    
-    def getTitle(self, value):
-        """Get a user-presentable description for a theme.
-        """
-        themes_provider = getUtility(IThemesProvider, name="default")
-        for name, theme in themes_provider():
-            if name == value:
-                return theme.description
-        return u'unnamed'
-
-class ThemesSourceBinder(object):
-    """A source binder for the `ThemeSource`.
-
-    Normally, we can pass instances of a source (like `ThemeSource`)
-    directly to fields in interfaces (writing something like ``source
-    = ThemeSource()``. With `ThemeSource` we cannot do this, because
-    the set of available themes is not available before the whole
-    `waeup.sirp` package was grokked (and parsed).
-
-    We bypass that problem by defining this trivial source-binder,
-    which is asked for themes only when the values of the source are
-    really needed and after the process of grokking and parsing
-    everything on startup.
-
-    So, although this binder binds to some context (`IUniversity` is
-    currently the only interface making use of this source, so
-    `University` instances are the only context used), the context
-    does not really matter.
-    """
-    implements(schema.interfaces.IContextSourceBinder)
-
-    def __call__(self, context):
-        return ThemesSource()
 
 class FatalCSVError(Exception):
@@ -92 +26 @@
     return SimpleVocabulary([
             SimpleTerm(value, value, title) for title, value in terms])
-
 
 class IWAeUPObject(Interface):
@@ -113 +46 @@
         title = u'Skin',
         default = u'waeuptheme-gray1.css',
-        source = ThemesSourceBinder(),
+        vocabulary = 'waeup.sirp.browser.theming.ThemesVocabulary',
         required = True,
         )