Big refactoring of App + Experimental support for color.json

.gitignore

@@ -6,3 +6,4 @@ frontend_app_html-document/dist/html-document.app.js
 frontend_lib/dist/tracim_frontend_lib.js
 npm-debug.log
 package-lock.json
+color.json

backend/README.md

@@ -81,6 +81,10 @@ create wsgidav configuration file for webdav:
 
     cp wsgidav.conf.sample wsgidav.conf
 
+if not did before, you need to create a color.json file at root of tracim_v2 :
+   
+    cp ../color.json.sample ../color.json
+
 ## Run Tracim_backend ##
 
 ### With Uwsgi ###

backend/development.ini.sample

@@ -193,6 +193,13 @@ frontend.serve = True
 # organisation.
 # frontend.dist_folder_path = /home/user/tracim_v2/frontend/dist
 
+### Color
+# check for color.json file in tracim_v2, check by default in tracim_v2 parent
+# dir of backend.
+# you can set a specific file path here
+# color.config_file_path = /home/user/tracim_v2/color.json
+
+
 ###
 # wsgi server configuration
 ###

backend/tracim_backend/app_models/applications.py

@@ -0,0 +1,71 @@
+# coding=utf-8
+import typing
+
+from tracim_backend.app_models.contents import ContentType
+
+
+class Application(object):
+    """
+    Application class with data needed for frontend
+    """
+    def __init__(
+            self,
+            label: str,
+            slug: str,
+            fa_icon: str,
+            is_active: bool,
+            config: typing.Dict[str, str],
+            main_route: str,
+            app_config: 'CFG',
+    ) -> None:
+        """
+        @param label: public label of application
+        @param slug: identifier of application
+        @param fa_icon: font awesome icon class
+        @param is_active: True if application enable, False if inactive
+        @param config: a dict with eventual application config
+        @param main_route: the route of the frontend "home" screen of
+        the application. For exemple, if you have an application
+        called "calendar", the main route will be something
+        like /#/workspace/{wid}/calendar.
+        """
+        self.label = label
+        self.slug = slug
+        self.fa_icon = fa_icon
+        self.hexcolor = self._get_hexcolor_or_default(slug, app_config)
+        self.is_active = is_active
+        self.config = config
+        self.main_route = main_route
+        self.content_types = []
+
+    # TODO - G.M - 2018-08-07 - Refactor slug coherence issue like this one.
+    # we probably should not have 2 kind of slug
+    @property
+    def minislug(self):
+        return self.slug.replace('contents/', '')
+
+    def add_content_type(
+            self,
+            label: str,
+            slug: str,
+            creation_label: str,
+            available_statuses: typing.List['ContentStatus'],
+            slug_alias: typing.List[str] = None,
+    ):
+        content_type = ContentType(
+            slug=slug,
+            fa_icon=self.fa_icon,
+            label=label,
+            hexcolor=self.hexcolor,
+            creation_label=creation_label,
+            available_statuses=available_statuses,
+            slug_alias=slug_alias,
+        )
+        self.content_types.append(content_type)
+
+    def _get_hexcolor_or_default(self, slug: str, app_config: 'CFG') -> str:
+        assert app_config.APPS_COLORS
+        assert 'primary' in app_config.APPS_COLORS
+        if slug in app_config.APPS_COLORS:
+            return app_config.APPS_COLORS[slug]
+        return app_config.APPS_COLORS['primary']

backend/tracim_backend/models/contents.py → backend/tracim_backend/app_models/contents.py

@@ -2,17 +2,13 @@
 import typing
 from enum import Enum
 
+from tracim_backend.extensions import APP_LIST
 from tracim_backend.exceptions import ContentTypeNotExist
 from tracim_backend.exceptions import ContentStatusNotExist
-from tracim_backend.models.applications import html_documents
-from tracim_backend.models.applications import _file
-from tracim_backend.models.applications import folder
-from tracim_backend.models.applications import thread
-from tracim_backend.models.applications import markdownpluspage
-
 
 ####
 # Content Status
+from tracim_backend.lib.core.application import ApplicationApi
 
 
 class GlobalStatus(Enum):
@@ -132,59 +128,17 @@ class ContentType(object):
         self.slug_alias = slug_alias
 
 
-thread_type = ContentType(
-    slug='thread',
-    fa_icon=thread.fa_icon,
-    hexcolor=thread.hexcolor,
-    label='Thread',
-    creation_label='Discuss about a topic',
-    available_statuses=CONTENT_STATUS.get_all(),
-)
-
-file_type = ContentType(
-    slug='file',
-    fa_icon=_file.fa_icon,
-    hexcolor=_file.hexcolor,
-    label='File',
-    creation_label='Upload a file',
-    available_statuses=CONTENT_STATUS.get_all(),
-)
-
-markdownpluspage_type = ContentType(
-    slug='markdownpage',
-    fa_icon=markdownpluspage.fa_icon,
-    hexcolor=markdownpluspage.hexcolor,
-    label='Rich Markdown File',
-    creation_label='Create a Markdown document',
-    available_statuses=CONTENT_STATUS.get_all(),
-)
-
-html_documents_type = ContentType(
-    slug='html-document',
-    fa_icon=html_documents.fa_icon,
-    hexcolor=html_documents.hexcolor,
-    label='Text Document',
-    creation_label='Write a document',
-    available_statuses=CONTENT_STATUS.get_all(),
-    slug_alias=['page']
-)
-
-# TODO - G.M - 31-05-2018 - Set Better folder params
-folder_type = ContentType(
-    slug='folder',
-    fa_icon=folder.fa_icon,
-    hexcolor=folder.hexcolor,
-    label='Folder',
-    creation_label='Create a folder',
-    available_statuses=CONTENT_STATUS.get_all(),
-)
-
+thread_type = 'thread'
+file_type = 'file'
+markdownpluspage_type = 'markdownpage'
+html_documents_type = 'html-document'
+folder_type = 'folder'
 
 # TODO - G.M - 31-05-2018 - Set Better Event params
 event_type = ContentType(
     slug='event',
-    fa_icon=thread.fa_icon,
-    hexcolor=thread.hexcolor,
+    fa_icon='',
+    hexcolor='',
     label='Event',
     creation_label='Event',
     available_statuses=CONTENT_STATUS.get_all(),
@@ -193,8 +147,8 @@ event_type = ContentType(
 # TODO - G.M - 31-05-2018 - Set Better Event params
 comment_type = ContentType(
     slug='comment',
-    fa_icon=thread.fa_icon,
-    hexcolor=thread.hexcolor,
+    fa_icon='',
+    hexcolor='',
     label='Comment',
     creation_label='Comment',
     available_statuses=CONTENT_STATUS.get_all(),
@@ -206,19 +160,37 @@ class ContentTypeList(object):
     ContentType List
     """
     Any_SLUG = 'any'
-    Folder = folder_type
     Comment = comment_type
     Event = event_type
-    File = file_type
-    Page = html_documents_type
-    Thread = thread_type
 
-    def __init__(self, extend_content_status: typing.List[ContentType]):
-        self._content_types = [self.Folder]
-        self._content_types.extend(extend_content_status)
+    @property
+    def Folder(self):
+        return self.get_one_by_slug('folder')
+
+    @property
+    def File(self):
+        return self.get_one_by_slug('file')
+
+    @property
+    def Page(self):
+        return self.get_one_by_slug('html-document')
+
+    @property
+    def Thread(self):
+        return self.get_one_by_slug('thread')
+
+    def __init__(self, app_list: typing.List['Application']):
+        self.app_list = app_list
         self._special_contents_types = [self.Comment]
         self._extra_slugs = [self.Any_SLUG]
 
+    @property
+    def _content_types(self):
+        app_api = ApplicationApi(self.app_list)
+        content_types = app_api.get_content_types()
+        content_types.extend(self._special_contents_types)
+        return content_types
+
     def get_one_by_slug(self, slug: str) -> ContentType:
         """
         Get ContentType object according to slug
@@ -258,12 +230,4 @@ class ContentTypeList(object):
         return allowed_types_slug
 
 
-CONTENT_TYPES = ContentTypeList(
-    [
-        thread_type,
-        file_type,
-        # TODO - G.M - 2018-08-02 - Restore markdown page content
-        #    markdownpluspage_type,
-        html_documents_type,
-    ]
-)
+CONTENT_TYPES = ContentTypeList(APP_LIST)

backend/tracim_backend/app_models/validator.py

@@ -0,0 +1,11 @@
+from marshmallow.validate import OneOf
+from tracim_backend.app_models.contents import CONTENT_TYPES
+
+# TODO - G.M - 2018-08-08 - [GlobalVar] Refactor Global var
+# of tracim_backend, Be careful ALL_CONTENT_TYPES_VALIDATOR is a global_var !
+
+ALL_CONTENT_TYPES_VALIDATOR = OneOf(choices=[])
+
+
+def update_validators():
+    ALL_CONTENT_TYPES_VALIDATOR.choices = CONTENT_TYPES.endpoint_allowed_types_slug()  # nopep8

backend/tracim_backend/app_models/workspace_menu_entries.py

@@ -0,0 +1,33 @@
+class WorkspaceMenuEntry(object):
+    """
+    Application class with data needed for frontend
+    """
+    def __init__(
+            self,
+            label: str,
+            slug: str,
+            fa_icon: str,
+            hexcolor: str,
+            route: str,
+    ) -> None:
+        self.slug = slug
+        self.label = label
+        self.route = route
+        self.hexcolor = hexcolor
+        self.fa_icon = fa_icon
+
+
+dashboard_menu_entry = WorkspaceMenuEntry(
+  slug='dashboard',
+  label='Dashboard',
+  route='/#/workspaces/{workspace_id}/dashboard',
+  hexcolor='#252525',
+  fa_icon="signal",
+)
+all_content_menu_entry = WorkspaceMenuEntry(
+  slug="contents/all",
+  label="All Contents",
+  route="/#/workspaces/{workspace_id}/contents",
+  hexcolor="#fdfdfd",
+  fa_icon="th",
+)

backend/tracim_backend/config.py

@@ -1,15 +1,19 @@
 # -*- coding: utf-8 -*-
+import json
 from urllib.parse import urlparse
 
 import os
 from paste.deploy.converters import asbool
+from tracim_backend.app_models.validator import update_validators
+from tracim_backend.extensions import APP_LIST
 from tracim_backend.lib.utils.logger import logger
 from depot.manager import DepotManager
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.applications import Application
+from tracim_backend.app_models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_STATUS
 from tracim_backend.models.data import ActionDescription
 
 
-
 class CFG(object):
     """Object used for easy access to config file parameters."""
 
@@ -41,7 +45,36 @@ class CFG(object):
         ###
         # General
         ###
+        backend_folder = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))  # nopep8
+        tracim_v2_folder = os.path.dirname(backend_folder)
+        default_color_config_file_path = os.path.join(tracim_v2_folder, 'color.json')  # nopep8
+        self.COLOR_CONFIG_FILE_PATH = settings.get(
+            'color.config_file_path', default_color_config_file_path
+        )
+        if not os.path.exists(self.COLOR_CONFIG_FILE_PATH):
+            raise Exception(
+                'ERROR: {} file does not exist. '
+                'please create it or set color.config_file_path'
+                'with a correct value'.format(self.COLOR_CONFIG_FILE_PATH)
+            )
+
+        try:
+            with open(self.COLOR_CONFIG_FILE_PATH) as json_file:
+                self.APPS_COLORS = json.load(json_file)
+        except Exception as e:
+            raise Exception(
+                'Error: {} file could not be load as json'.format(self.COLOR_CONFIG_FILE_PATH) # nopep8
+            ) from e
 
+        try:
+            self.APPS_COLORS['primary']
+        except KeyError as e:
+            raise Exception(
+                'Error: primary color is required in {} file'.format(
+                    self.COLOR_CONFIG_FILE_PATH)  # nopep8
+            ) from e
+
+        self._set_default_app()
         mandatory_msg = \
             'ERROR: {} configuration is mandatory. Set it before continuing.'
         self.DEPOT_STORAGE_DIR = settings.get(
@@ -450,8 +483,6 @@ class CFG(object):
         # INFO - G.M - 2018-08-06 - we pretend that frontend_dist_folder
         # is probably in frontend subfolder
         # of tracim_v2 parent of both backend and frontend
-        backend_folder = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))  # nopep8
-        tracim_v2_folder = os.path.dirname(backend_folder)
         frontend_dist_folder = os.path.join(tracim_v2_folder, 'frontend', 'dist')  # nopep8
 
         self.FRONTEND_DIST_FOLDER_PATH = settings.get(
@@ -467,6 +498,10 @@ class CFG(object):
             )
 
     def configure_filedepot(self):
+
+        # TODO - G.M - 2018-08-08 - [GlobalVar] Refactor Global var
+        # of tracim_backend, Be careful DepotManager is a Singleton !
+
         depot_storage_name = self.DEPOT_STORAGE_NAME
         depot_storage_path = self.DEPOT_STORAGE_DIR
         depot_storage_settings = {'depot.storage_path': depot_storage_path}
@@ -475,6 +510,115 @@ class CFG(object):
             depot_storage_settings,
         )
 
+    def _set_default_app(self):
+        calendar = Application(
+            label='Calendar',
+            slug='calendar',
+            fa_icon='calendar',
+            is_active=False,
+            config={},
+            main_route='/#/workspaces/{workspace_id}/calendar',
+            app_config=self
+        )
+
+        thread = Application(
+            label='Threads',
+            slug='contents/thread',
+            fa_icon='comments-o',
+            is_active=True,
+            config={},
+            main_route='/#/workspaces/{workspace_id}/contents?type=thread',
+            app_config=self
+        )
+        thread.add_content_type(
+            slug='thread',
+            label='Thread',
+            creation_label='Discuss about a topic',
+            available_statuses=CONTENT_STATUS.get_all(),
+        )
+
+        folder = Application(
+            label='Folder',
+            slug='contents/folder',
+            fa_icon='folder-open-o',
+            is_active=True,
+            config={},
+            main_route='',
+            app_config=self
+        )
+        folder.add_content_type(
+            slug='folder',
+            label='Folder',
+            creation_label='Create a folder',
+            available_statuses=CONTENT_STATUS.get_all(),
+        )
+
+        _file = Application(
+            label='Files',
+            slug='contents/file',
+            fa_icon='paperclip',
+            is_active=True,
+            config={},
+            main_route='/#/workspaces/{workspace_id}/contents?type=file',
+            app_config=self,
+        )
+        _file.add_content_type(
+            slug='file',
+            label='File',
+            creation_label='Upload a file',
+            available_statuses=CONTENT_STATUS.get_all(),
+        )
+
+        markdownpluspage = Application(
+            label='Markdown Plus Documents',
+            # TODO - G.M - 24-05-2018 - Check label
+            slug='content/markdownpluspage',
+            fa_icon='file-code-o',
+            is_active=False,
+            config={},
+            main_route='/#/workspaces/{workspace_id}/contents?type=markdownpluspage',
+            # nopep8
+            app_config=self,
+        )
+        markdownpluspage.add_content_type(
+            slug='markdownpage',
+            label='Rich Markdown File',
+            creation_label='Create a Markdown document',
+            available_statuses=CONTENT_STATUS.get_all(),
+        )
+
+        html_documents = Application(
+            label='Text Documents',  # TODO - G.M - 24-05-2018 - Check label
+            slug='contents/html-document',
+            fa_icon='file-text-o',
+            is_active=True,
+            config={},
+            main_route='/#/workspaces/{workspace_id}/contents?type=html-document',
+            app_config=self
+        )
+        html_documents.add_content_type(
+            slug='html-document',
+            label='Text Document',
+            creation_label='Write a document',
+            available_statuses=CONTENT_STATUS.get_all(),
+            slug_alias=['page']
+        )
+
+        # TODO - G.M - 2018-08-08 - [GlobalVar] Refactor Global var
+        # of tracim_backend, Be careful APP_LIST is a global_var
+        APP_LIST.clear()
+        APP_LIST.extend([
+            html_documents,
+            markdownpluspage,
+            _file,
+            thread,
+            folder,
+            calendar,
+        ])
+        # TODO - G.M - 2018-08-08 - We need to update validators each time
+        # APP_LIST is updated.
+        update_validators()
+
     class CST(object):
         ASYNC = 'ASYNC'
         SYNC = 'SYNC'

backend/tracim_backend/exceptions.py

@@ -211,3 +211,7 @@ class TooShortAutocompleteString(TracimException):
 
 class PageNotFound(TracimException):
     pass
+
+
+class AppDoesNotExist(TracimException):
+    pass

backend/tracim_backend/extensions.py

@@ -1,3 +1,14 @@
 from hapic import Hapic
 
 hapic = Hapic()
+
+# TODO - G.M - 2018-08-08 - [GlobalVar] Refactor Global var of tracim_backend
+
+# INFO - G.M - 2018-08-08 - APP_LIST
+# APP_LIST is one of the few "global_val" in tracim_backend, with hapic
+# and ALL_CONTENT_TYPES_VALIDATOR.
+# The goal of this is to be able to get current list of loaded app.
+# List is empty until config load apps.
+# If you need to update APP_LIST, think about updating Content validator like
+# ALL_CONTENT_TYPES_VALIDATOR , see  update_validators() method.
+APP_LIST = []

backend/tracim_backend/fixtures/content.py

@@ -8,7 +8,7 @@ from tracim_backend.fixtures.users_and_groups import Test
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.userworkspace import RoleApi
 from tracim_backend.lib.core.workspace import WorkspaceApi
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.data import UserRoleInWorkspace
 from tracim_backend.models.revision_protection import new_revision
 

backend/tracim_backend/lib/core/application.py

@@ -0,0 +1,70 @@
+import typing
+from copy import copy
+
+from tracim_backend.exceptions import AppDoesNotExist
+from tracim_backend.app_models.workspace_menu_entries import WorkspaceMenuEntry
+from tracim_backend.app_models.workspace_menu_entries import dashboard_menu_entry
+from tracim_backend.app_models.workspace_menu_entries import all_content_menu_entry
+
+
+class ApplicationApi(object):
+
+    def __init__(
+        self,
+        app_list,
+        show_all: bool = False,
+    ) ->  None:
+        self.apps = app_list
+        self.show_all = show_all
+
+    def get_one(self, slug):
+        for app in self.apps:
+            if app.slug == slug:
+                return app
+        raise AppDoesNotExist('Application {app} does not exist'.format(app=slug))  # nopep8
+
+    def get_all(self):
+        active_apps = []
+        for app in self.apps:
+            if self.show_all or app.is_active:
+                active_apps.append(app)
+
+        return active_apps
+
+    def get_content_types(self):
+        active_content_types = []
+        for app in self.get_all():
+            if app.content_types:
+                for content_type in app.content_types:
+                    active_content_types.append(content_type)
+        return active_content_types
+
+    def get_default_workspace_menu_entry(
+            self,
+            workspace: 'Workspace',
+    ) -> typing.List[WorkspaceMenuEntry]:
+        """
+        Get default menu entry for a workspace
+        """
+        menu_entries = [
+            copy(dashboard_menu_entry),
+            copy(all_content_menu_entry),
+        ]
+        for app in self.get_all():
+            if app.main_route:
+                new_entry = WorkspaceMenuEntry(
+                    slug=app.slug,
+                    label=app.label,
+                    hexcolor=app.hexcolor,
+                    fa_icon=app.fa_icon,
+                    route=app.main_route
+                )
+                menu_entries.append(new_entry)
+
+        for entry in menu_entries:
+            entry.route = entry.route.replace(
+                '{workspace_id}',
+                str(workspace.workspace_id)
+            )
+
+        return menu_entries

backend/tracim_backend/lib/core/content.py

@@ -34,9 +34,9 @@ from tracim_backend.exceptions import EmptyLabelNotAllowed
 from tracim_backend.exceptions import ContentNotFound
 from tracim_backend.exceptions import WorkspacesDoNotMatch
 from tracim_backend.lib.utils.utils import current_date_for_filename
-from tracim_backend.models.contents import CONTENT_STATUS
-from tracim_backend.models.contents import ContentType
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_STATUS
+from tracim_backend.app_models.contents import ContentType
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.revision_protection import new_revision
 from tracim_backend.models.auth import User
 from tracim_backend.models.data import ActionDescription

backend/tracim_backend/lib/mail_notifier/notifier.py

@@ -20,7 +20,7 @@ from tracim_backend.lib.utils.logger import logger
 from tracim_backend.lib.utils.utils import get_login_frontend_url
 from tracim_backend.lib.utils.utils import get_email_logo_frontend_url
 from tracim_backend.models.auth import User
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.context_models import ContentInContext
 from tracim_backend.models.context_models import WorkspaceInContext
 from tracim_backend.models.data import ActionDescription

backend/tracim_backend/lib/utils/authentification.py

@@ -3,7 +3,7 @@ import typing
 from pyramid.request import Request
 from sqlalchemy.orm.exc import NoResultFound
 
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.exceptions import UserDoesNotExist
 from tracim_backend.lib.core.user import UserApi
 from tracim_backend.models import User

backend/tracim_backend/lib/utils/authorization.py

@@ -5,15 +5,14 @@ import functools
 from pyramid.interfaces import IAuthorizationPolicy
 from zope.interface import implementer
 
-from tracim_backend.models.contents import ContentType
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import ContentType
+from tracim_backend.app_models.contents import CONTENT_TYPES
 
 try:
     from json.decoder import JSONDecodeError
 except ImportError:  # python3.4
     JSONDecodeError = ValueError
 
-from tracim_backend.models.contents import ContentType
 from tracim_backend.exceptions import InsufficientUserRoleInWorkspace
 from tracim_backend.exceptions import ContentTypeNotAllowed
 from tracim_backend.exceptions import InsufficientUserProfile
@@ -133,19 +132,18 @@ def require_candidate_workspace_role(minimal_required_role: int) -> typing.Calla
     return decorator
 
 
-def require_content_types(content_types: typing.List['ContentType']) -> typing.Callable:  # nopep8
+def require_content_types(content_types_slug: typing.List[str]) -> typing.Callable:  # nopep8
     """
     Restricts access to specific file type or raise an exception.
     Check role for candidate_workspace.
-    :param content_types: list of ContentType object
+    :param content_types_slug: list of slug of content_types
     :return: decorator
     """
     def decorator(func: typing.Callable) -> typing.Callable:
         @functools.wraps(func)
         def wrapper(self, context, request: 'TracimRequest') -> typing.Callable:
             content = request.current_content
-            current_content_type_slug = CONTENT_TYPES.get_one_by_slug(content.type).slug
-            content_types_slug = [content_type.slug for content_type in content_types]  # nopep8
+            current_content_type_slug = CONTENT_TYPES.get_one_by_slug(content.type).slug  # nopep8
             if current_content_type_slug in content_types_slug:
                 return func(self, context, request)
             raise ContentTypeNotAllowed()

backend/tracim_backend/lib/utils/request.py

@@ -15,7 +15,7 @@ from tracim_backend.exceptions import UserNotFoundInTracimRequest
 from tracim_backend.exceptions import UserDoesNotExist
 from tracim_backend.exceptions import WorkspaceNotFound
 from tracim_backend.exceptions import ImmutableAttribute
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.user import UserApi
 from tracim_backend.lib.core.workspace import WorkspaceApi

backend/tracim_backend/lib/utils/utils.py

@@ -7,7 +7,6 @@ from enum import Enum
 from redis import Redis
 from rq import Queue
 
-from tracim_backend.config import CFG
 
 DATETIME_FORMAT = '%Y-%m-%dT%H:%M:%SZ'
 DEFAULT_WEBDAV_CONFIG_FILE = "wsgidav.conf"
@@ -16,7 +15,7 @@ CONTENT_FRONTEND_URL_SCHEMA = 'workspaces/{workspace_id}/contents/{content_type}
 WORKSPACE_FRONTEND_URL_SCHEMA = 'workspaces/{workspace_id}'  # nopep8
 
 
-def get_root_frontend_url(config: CFG) -> str:
+def get_root_frontend_url(config: 'CFG') -> str:
     """
     Return website base url with always '/' at the end
     """
@@ -28,19 +27,19 @@ def get_root_frontend_url(config: CFG) -> str:
     return base_url
 
 
-def get_login_frontend_url(config: CFG):
+def get_login_frontend_url(config: 'CFG'):
     """
     Return login page url
     """
     return get_root_frontend_url(config) + 'login'
 
 
-def get_email_logo_frontend_url(config: CFG):
+def get_email_logo_frontend_url(config: 'CFG'):
     # TODO - G.M - 11-06-2018 - [emailTemplateURL] correct value for email_logo_frontend_url  # nopep8
     return 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAADAAAAAwCAYAAABXAvmHAAAABmJLR0QA/wD/AP+gvaeTAAAACXBIWXMAAC4jAAAuIwF4pT92AAAAB3RJTUUH4QUTDjMSlsws9AAAB89JREFUaN7tmWtwlFcZx3/PeXeXXLmKQJEGsFZApIUBnaqjUm4VgdYyCUib0cJoKLmHW5EWmlGoQkMICURgEPuhtiYDU7XcyVgKVugULZVCQAwUYyENYAmEXHb3PH7IFjYQNgnZHfnA+bKz+7573v//PPf/C/fWvdWhJWHfMbO4P479OpCAigeowWglPq3ARWfU9AJ7Cb/3MAXZn95dBLKKHsfIQmAUIq7rv6t6gU+BKCAGaEBkN9h88tLe7sgjTdjAJ5Y4OGYMIo80Aw8g4kakJyLxiDiIxABPoMwj5+XP3R0ESpP8WC1H1YuqBj4bUWwIB5gA0d/6/xFQFUpKHBJLHADyU9chTEc1BdEZqJ0O+ofb4xcPyuBIxoBQuHc4ov3w2zrwH0Nd0aBDccwgVPsjEg/4Uc4jegKfvkfMmQ9JSfECkFP0HGJeuu0TrJZhSacg9Xh4Caze2wsXC1B9GugO+BAqUIkC/QIinhYsYoEqYDt+fY2s8WXkFE5CnBIg+vaWtJuQukzy5te2l4Bz2ysTkxchshCRWERMU2BKT4RuiLT0vwagHsENPIyR7zAx2eGybws09AUZEQLHcNS5wMEdB8NjgaKyBNTuQuTLoWOAs6AHUd5DpRzxX0KMBe0M+gA4D4Hdw5mT72LJQORxkM6IfgxSC3wjyHpHEN8M8jKPtYeAq2VgvhHg9A4RvPUIKzCyBVe346SM9LZw1y5KShzOxfUhL60SmMfcNUXgisXYS/iNA/IzhNmBgH4I3E8Cxzpmgby9ffFoMSKTQxA4jZjRpI35qENZbF5Rb1SWgcwM7HsQr28KhZnVd55GPfqTkOCbaB+i2lR1uHa8nHYeda0EygJWGILHGXzndaBgz3CQ5Fb8/hSqr5E7uj4sBXBVSjlqN6FaixKHlS8BxCYtHhs/bUly9JT597U9C30/eRYiU4Nc5Q1gH+AG/gvsxJBH2vg3w9pDPTjqFDGxX0FkKOih2PtHOcbIBiPMMi5XT9eDo9/xlu+rDW2BpX+MAQnOCudRWcWFA3OwncYhrke5VpuK1SrW7+sTVgKv5NYjsr2p6TMY1QEGvhi4mmw89unWXaiX6QQEN1YHuHjpELm5lsxvV2MbhxAdWwCSiNfnCXsb7mc/Ih8hXED8J1T1k6awEAG+R+LCLqEJOL2vInwQOP46VHaSm9QYiI1nMU4xwv2oa3OHs0+LiaHuHKr/wOo/r1yuOQwcDEqVw2KdqITQBFJGelFWojITPz8kY+wmAAp3pWLkJVT7oGwkY3R5REar/Ln1qO7AyIfsLGxQ1eB60E3UDmi9kKWPOwmcvP69cM8kROYCXVB5FW18I4LToeLSbTGnKh2T9MIChLHBOI2lT9srMUDB9s7AM8CApj7H/yaZExsiOd9GnbngcjzOciPyVPOuW0RFB7ZvHnDck4FJgbOpwPoPRHpAdxs782bwQSwei3ty0aC2EVh61ANMutEy63Gi/l0VUfSJC7sIjAnhYYON4x7eNgI9KgeifDPIOyuuDygRWnGuKDcQF6roWrGxbSNgzDAgqBuV6ki7z9WT5y4DR0LdoqoVbSOgZhAi7hv4bXzEFarDG7yqbPmsgN2aovhdLdf+2jqBorIE4JGbIugZCvcsijQHr5ojN1tbVaustUWNNP6S0vy60AR+ut4Ndj7CYzdlgL7AixTtmRnRLOToVNAhQeDfR+yPr+iJrIbf/+pM62l0WP8RKNNCSCCTySuJjgT42MTnv4rqjEDvg6J+VH5T8/qynZSW+ts40EgCQtcQDcsDEN2d9IKe4U2hiR5jmG1ERt4QKrS4pkqK2zmRSQMaUsj6D5cuu3C5ppCY6IQLf2dn8CIjZk4Q+N961VnJvlxf+4Z6v/cojvsDYEQL4BXRHVTXfkysjqPfd3tA6YqOnXx293gnLgvV5xFBVa2iv/b6dEX91p+fbf9ImTnxX6gWAadbILARX+0rbEjxohwCFpBVlH1nyJea2KTFY+OduHUGeaFJoNN3gcwr1smu37qsze16y7pQ4a7pYFYj0iuAvho0jbTxJQBkrB6Ky70VGIDqamA9+WmnWtwrZ20/fN4uQA1rss52Slw8wIOZKg4/QIlTKBfY32DttobSZafDJS0Ka3YXYEx6UCXZid8/n6wJRwHIXrsEI7kB6+xHtQTDn8hLPQuipBb2oJOkgSQBPREuAlupq98cfa6mXsXnOFpfV5uQMBRxlMpP/kJpbmP4tNHC3SNBNiLycJAbleLlOXLGVTB33edRLUBkepAuegyRd1D7Pphh10Wr5mJuMT5fNo67L45NQSUlIBqsYlXqkqaiGy51es3eWQh5CDfmUeufTcaE9QBkFY7AyIuImXxT3bfI7doUvQayFbUDMSZYRHiLq64n2JByOXzvBzLGbkJ1GarnP4OPmBt60Or0v+E381Fdj+ILOhYT4p1ADOhTzcA3EdjeXvCtEwDIGNc0J6vNB/0FDXXbmk9uc05wMSYLq+nAgYArtWJ3keYW0eVUVheE9/3ArbYXkND+mbO2H8gk0AnAlGZAb3WlWpQ/I/o6q9JevWMxIyJdWWZ+V4xnOUaebQH4MVS2I/59eO3bFGbWdEiNiVhrmVncH+NfhMg0IA4RH1bfQnQlq9LKwiYnRbTB/9HmKLrWfg2j94HUoI1/Z3XOOe6te+vuWf8DkM0cb7DOQZgAAAAASUVORK5CYII='  # nopep8'
 
 
-def get_redis_connection(config: CFG) -> Redis:
+def get_redis_connection(config: 'CFG') -> Redis:
     """
     :param config: current app_config
     :return: redis connection

backend/tracim_backend/lib/webdav/dav_provider.py

@@ -8,7 +8,7 @@ from sqlalchemy.orm.exc import NoResultFound
 from tracim_backend import CFG
 from tracim_backend.lib.webdav.utils import transform_to_bdd, HistoryType, \
     SpecialFolderExtension
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 
 from wsgidav.dav_provider import DAVProvider
 from wsgidav.lock_manager import LockManager

backend/tracim_backend/lib/webdav/design.py

@@ -1,7 +1,7 @@
 #coding: utf8
 from datetime import datetime
 
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.data import VirtualEvent
 from tracim_backend.models import data
 

backend/tracim_backend/lib/webdav/resources.py

@@ -19,7 +19,7 @@ from tracim_backend.lib.webdav.utils import transform_to_display, HistoryType, \
     FakeFileStream
 from tracim_backend.lib.webdav.utils import transform_to_bdd
 from tracim_backend.lib.core.workspace import WorkspaceApi
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.data import User, ContentRevisionRO
 from tracim_backend.models.data import Workspace
 from tracim_backend.models.data import Content

backend/tracim_backend/lib/webdav/utils.py

@@ -4,7 +4,7 @@ import transaction
 from os.path import normpath as base_normpath
 
 from sqlalchemy.orm import Session
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from wsgidav import util
 from wsgidav import compat
 

backend/tracim_backend/models/__init__.py

@@ -5,7 +5,7 @@ from sqlalchemy.orm import sessionmaker
 from sqlalchemy.orm import configure_mappers
 import zope.sqlalchemy
 from .meta import DeclarativeBase
-from .revision_protection import prevent_content_revision_delete
+from tracim_backend.models.revision_protection import prevent_content_revision_delete
 # import or define all models here to ensure they are attached to the
 # Base.metadata prior to any initialization routines
 from tracim_backend.models.auth import User, Group, Permission

backend/tracim_backend/models/applications.py

@@ -1,117 +0,0 @@
-# coding=utf-8
-import typing
-
-
-class Application(object):
-    """
-    Application class with data needed for frontend
-    """
-    def __init__(
-            self,
-            label: str,
-            slug: str,
-            fa_icon: str,
-            hexcolor: str,
-            is_active: bool,
-            config: typing.Dict[str, str],
-            main_route: str,
-    ) -> None:
-        """
-        @param label: public label of application
-        @param slug: identifier of application
-        @param icon: font awesome icon class
-        @param hexcolor: hexa color of application main color
-        @param is_active: True if application enable, False if inactive
-        @param config: a dict with eventual application config
-        @param main_route: the route of the frontend "home" screen of
-        the application. For exemple, if you have an application
-        called "calendar", the main route will be something
-        like /#/workspace/{wid}/calendar.
-        """
-        self.label = label
-        self.slug = slug
-        self.fa_icon = fa_icon
-        self.hexcolor = hexcolor
-        self.is_active = is_active
-        self.config = config
-        self.main_route = main_route
-
-    # TODO - G.M - 2018-08-07 - Refactor slug coherence issue like this one.
-    # we probably should not have 2 kind of slug
-    @property
-    def minislug(self):
-        return self.slug.replace('contents/', '')
-
-
-# default apps
-calendar = Application(
-    label='Calendar',
-    slug='calendar',
-    fa_icon='calendar',
-    hexcolor='#757575',
-    is_active=True,
-    config={},
-    main_route='/#/workspaces/{workspace_id}/calendar',
-)
-
-thread = Application(
-    label='Threads',
-    slug='contents/thread',
-    fa_icon='comments-o',
-    hexcolor='#ad4cf9',
-    is_active=True,
-    config={},
-    main_route='/#/workspaces/{workspace_id}/contents?type=thread',
-
-)
-
-folder = Application(
-    label='Folder',
-    slug='contents/folder',
-    fa_icon='folder-open-o',
-    hexcolor='#252525',
-    is_active=True,
-    config={},
-    main_route='',
-)
-
-_file = Application(
-    label='Files',
-    slug='contents/file',
-    fa_icon='paperclip',
-    hexcolor='#FF9900',
-    is_active=True,
-    config={},
-    main_route='/#/workspaces/{workspace_id}/contents?type=file',
-)
-
-markdownpluspage = Application(
-    label='Markdown Plus Documents',  # TODO - G.M - 24-05-2018 - Check label
-    slug='contents/markdownpluspage',
-    fa_icon='file-code-o',
-    hexcolor='#f12d2d',
-    is_active=True,
-    config={},
-    main_route='/#/workspaces/{workspace_id}/contents?type=markdownpluspage',
-)
-
-html_documents = Application(
-    label='Text Documents',  # TODO - G.M - 24-05-2018 - Check label
-    slug='contents/html-document',
-    fa_icon='file-text-o',
-    hexcolor='#3f52e3',
-    is_active=True,
-    config={},
-    main_route='/#/workspaces/{workspace_id}/contents?type=html-document',
-)
-# TODO - G.M - 08-06-2018 - This is hardcoded lists of app, make this dynamic.
-# List of applications
-applications = [
-    html_documents,
-    # TODO - G.M - 2018-08-02 - Restore markdownpage app
-    # markdownpluspage,
-    _file,
-    thread,
-    folder,
-    # calendar,
-]

backend/tracim_backend/models/context_models.py

@@ -7,6 +7,8 @@ from slugify import slugify
 from sqlalchemy.orm import Session
 from tracim_backend.config import CFG
 from tracim_backend.config import PreviewDim
+from tracim_backend.extensions import APP_LIST
+from tracim_backend.lib.core.application import ApplicationApi
 from tracim_backend.lib.utils.utils import get_root_frontend_url
 from tracim_backend.lib.utils.utils import CONTENT_FRONTEND_URL_SCHEMA
 from tracim_backend.lib.utils.utils import WORKSPACE_FRONTEND_URL_SCHEMA
@@ -17,9 +19,8 @@ from tracim_backend.models.data import ContentRevisionRO
 from tracim_backend.models.data import Workspace
 from tracim_backend.models.data import UserRoleInWorkspace
 from tracim_backend.models.roles import WorkspaceRoles
-from tracim_backend.models.workspace_menu_entries import default_workspace_menu_entry  # nopep8
-from tracim_backend.models.workspace_menu_entries import WorkspaceMenuEntry
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.workspace_menu_entries import WorkspaceMenuEntry
+from tracim_backend.app_models.contents import CONTENT_TYPES
 
 
 class PreviewAllowedDim(object):
@@ -463,7 +464,10 @@ class WorkspaceInContext(object):
         # order to not use hardcoded list
         # list should be able to change (depending on activated/disabled
         # apps)
-        return default_workspace_menu_entry(self.workspace)
+        app_api = ApplicationApi(
+            APP_LIST
+        )
+        return app_api.get_default_workspace_menu_entry(self.workspace)
 
     @property
     def frontend_url(self):

backend/tracim_backend/models/data.py

@@ -291,9 +291,9 @@ class ActionDescription(object):
                 ]
 
 
-from tracim_backend.models.contents import CONTENT_STATUS
-from tracim_backend.models.contents import ContentStatus
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_STATUS
+from tracim_backend.app_models.contents import ContentStatus
+from tracim_backend.app_models.contents import CONTENT_TYPES
 # TODO - G.M - 30-05-2018 - Drop this old code when whe are sure nothing
 # is lost .
 

backend/tracim_backend/models/workspace_menu_entries.py

@@ -1,71 +0,0 @@
-# coding=utf-8
-import typing
-from copy import copy
-
-from tracim_backend.models.applications import applications
-from tracim_backend.models.data import Workspace
-
-
-class WorkspaceMenuEntry(object):
-    """
-    Application class with data needed for frontend
-    """
-    def __init__(
-            self,
-            label: str,
-            slug: str,
-            fa_icon: str,
-            hexcolor: str,
-            route: str,
-    ) -> None:
-        self.slug = slug
-        self.label = label
-        self.route = route
-        self.hexcolor = hexcolor
-        self.fa_icon = fa_icon
-
-dashboard_menu_entry = WorkspaceMenuEntry(
-  slug='dashboard',
-  label='Dashboard',
-  route='/#/workspaces/{workspace_id}/dashboard',
-  hexcolor='#252525',
-  fa_icon="signal",
-)
-all_content_menu_entry = WorkspaceMenuEntry(
-  slug="contents/all",
-  label="All Contents",
-  route="/#/workspaces/{workspace_id}/contents",
-  hexcolor="#fdfdfd",
-  fa_icon="th",
-)
-
-# TODO - G.M - 08-06-2018 - This is hardcoded default menu entry,
-#  of app, make this dynamic (and loaded from application system)
-def default_workspace_menu_entry(
-    workspace: Workspace,
-)-> typing.List[WorkspaceMenuEntry]:
-    """
-    Get default menu entry for a workspace
-    """
-    menu_entries = [
-        copy(dashboard_menu_entry),
-        copy(all_content_menu_entry),
-    ]
-    for app in applications:
-        if app.main_route:
-            new_entry = WorkspaceMenuEntry(
-                slug=app.slug,
-                label=app.label,
-                hexcolor=app.hexcolor,
-                fa_icon=app.fa_icon,
-                route=app.main_route
-            )
-            menu_entries.append(new_entry)
-
-    for entry in menu_entries:
-        entry.route = entry.route.replace(
-            '{workspace_id}',
-            str(workspace.workspace_id)
-        )
-
-    return menu_entries

backend/tracim_backend/tests/__init__.py

@@ -14,7 +14,7 @@ from tracim_backend.models import get_engine
 from tracim_backend.models import DeclarativeBase
 from tracim_backend.models import get_session_factory
 from tracim_backend.models import get_tm_session
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.data import Workspace
 from tracim_backend.models.data import ContentRevisionRO
 from tracim_backend.models.data import Content

backend/tracim_backend/tests/functional/test_contents.py

@@ -5,7 +5,7 @@ from tracim_backend import models
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.workspace import WorkspaceApi
 from tracim_backend.models import get_tm_session
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.revision_protection import new_revision
 import io
 

backend/tracim_backend/tests/functional/test_mail_notification.py

@@ -11,7 +11,7 @@ from tracim_backend.fixtures.users_and_groups import Base as BaseFixture
 from tracim_backend.fixtures.content import Content as ContentFixture
 from tracim_backend.lib.utils.utils import get_redis_connection
 from tracim_backend.lib.utils.utils import get_rq_queue
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.user import UserApi

backend/tracim_backend/tests/functional/test_system.py

@@ -1,7 +1,10 @@
 # coding=utf-8
-from tracim_backend.models.contents import CONTENT_TYPES
+import transaction
+from tracim_backend.extensions import APP_LIST
+from tracim_backend.lib.core.application import ApplicationApi
+from tracim_backend.models import get_tm_session
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.tests import FunctionalTest
-from tracim_backend.models.applications import applications
 
 """
 Tests for /api/v2/system subpath endpoints.
@@ -26,6 +29,11 @@ class TestApplicationEndpoint(FunctionalTest):
         )
         res = self.testapp.get('/api/v2/system/applications', status=200)
         res = res.json_body
+        dbsession = get_tm_session(self.session_factory, transaction.manager)
+        app_api = ApplicationApi(
+            app_list=APP_LIST,
+        )
+        applications = app_api.get_all()
         assert len(res) == len(applications)
         for counter, application in enumerate(applications):
             assert res[counter]['label'] == application.label

backend/tracim_backend/tests/functional/test_user.py

@@ -7,13 +7,15 @@ import pytest
 import transaction
 
 from tracim_backend import models
+from tracim_backend.extensions import APP_LIST
+from tracim_backend.lib.core.application import ApplicationApi
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.user import UserApi
 from tracim_backend.lib.core.group import GroupApi
 from tracim_backend.lib.core.userworkspace import RoleApi
 from tracim_backend.lib.core.workspace import WorkspaceApi
 from tracim_backend.models import get_tm_session
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.data import UserRoleInWorkspace
 from tracim_backend.models.revision_protection import new_revision
 from tracim_backend.tests import FunctionalTest
@@ -2369,6 +2371,22 @@ class TestUserWorkspaceEndpoint(FunctionalTest):
         """
         Check obtain all workspaces reachables for user with user auth.
         """
+        dbsession = get_tm_session(self.session_factory, transaction.manager)
+        admin = dbsession.query(models.User) \
+            .filter(models.User.email == 'admin@admin.admin') \
+            .one()
+
+        workspace_api = WorkspaceApi(
+            session=dbsession,
+            current_user=admin,
+            config=self.app_config,
+        )
+        workspace = workspace_api.get_one(1)
+        app_api = ApplicationApi(
+            APP_LIST
+        )
+
+        default_sidebar_entry = app_api.get_default_workspace_menu_entry(workspace=workspace)  # nope8
         self.testapp.authorization = (
             'Basic',
             (
@@ -2382,44 +2400,13 @@ class TestUserWorkspaceEndpoint(FunctionalTest):
         assert workspace['workspace_id'] == 1
         assert workspace['label'] == 'Business'
         assert workspace['slug'] == 'business'
-        assert len(workspace['sidebar_entries']) == 5
-
-        # TODO - G.M - 2018-08-02 - Better test for sidebar entry, make it
-        # not fixed on active application/content-file
-        sidebar_entry = workspace['sidebar_entries'][0]
-        assert sidebar_entry['slug'] == 'dashboard'
-        assert sidebar_entry['label'] == 'Dashboard'
-        assert sidebar_entry['route'] == '/#/workspaces/1/dashboard'  # nopep8
-        assert sidebar_entry['hexcolor'] == "#252525"
-        assert sidebar_entry['fa_icon'] == "signal"
-
-        sidebar_entry = workspace['sidebar_entries'][1]
-        assert sidebar_entry['slug'] == 'contents/all'
-        assert sidebar_entry['label'] == 'All Contents'
-        assert sidebar_entry['route'] == "/#/workspaces/1/contents"  # nopep8
-        assert sidebar_entry['hexcolor'] == "#fdfdfd"
-        assert sidebar_entry['fa_icon'] == "th"
-
-        sidebar_entry = workspace['sidebar_entries'][2]
-        assert sidebar_entry['slug'] == 'contents/html-document'
-        assert sidebar_entry['label'] == 'Text Documents'
-        assert sidebar_entry['route'] == '/#/workspaces/1/contents?type=html-document'  # nopep8
-        assert sidebar_entry['hexcolor'] == "#3f52e3"
-        assert sidebar_entry['fa_icon'] == "file-text-o"
-
-        sidebar_entry = workspace['sidebar_entries'][3]
-        assert sidebar_entry['slug'] == 'contents/file'
-        assert sidebar_entry['label'] == 'Files'
-        assert sidebar_entry['route'] == "/#/workspaces/1/contents?type=file"  # nopep8
-        assert sidebar_entry['hexcolor'] == "#FF9900"
-        assert sidebar_entry['fa_icon'] == "paperclip"
-
-        sidebar_entry = workspace['sidebar_entries'][4]
-        assert sidebar_entry['slug'] == 'contents/thread'
-        assert sidebar_entry['label'] == 'Threads'
-        assert sidebar_entry['route'] == "/#/workspaces/1/contents?type=thread"  # nopep8
-        assert sidebar_entry['hexcolor'] == "#ad4cf9"
-        assert sidebar_entry['fa_icon'] == "comments-o"
+        assert len(workspace['sidebar_entries']) == len(default_sidebar_entry)
+        for counter, sidebar_entry in enumerate(default_sidebar_entry):
+            workspace['sidebar_entries'][counter]['slug'] = sidebar_entry.slug
+            workspace['sidebar_entries'][counter]['label'] = sidebar_entry.label
+            workspace['sidebar_entries'][counter]['route'] = sidebar_entry.route
+            workspace['sidebar_entries'][counter]['hexcolor'] = sidebar_entry.hexcolor  # nopep8
+            workspace['sidebar_entries'][counter]['fa_icon'] = sidebar_entry.fa_icon  # nopep8
 
 
     def test_api__get_user_workspaces__err_403__unallowed_user(self):

backend/tracim_backend/tests/functional/test_workspaces.py

@@ -7,10 +7,12 @@ import transaction
 from depot.io.utils import FileIntent
 
 from tracim_backend import models
+from tracim_backend.extensions import APP_LIST
+from tracim_backend.lib.core.application import ApplicationApi
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.workspace import WorkspaceApi
 from tracim_backend.models import get_tm_session
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.tests import FunctionalTest
 from tracim_backend.tests import set_html_document_slug_to_legacy
 from tracim_backend.fixtures.content import Content as ContentFixtures
@@ -28,6 +30,22 @@ class TestWorkspaceEndpoint(FunctionalTest):
         """
         Check obtain workspace reachable for user.
         """
+        dbsession = get_tm_session(self.session_factory, transaction.manager)
+        admin = dbsession.query(models.User) \
+            .filter(models.User.email == 'admin@admin.admin') \
+            .one()
+
+        workspace_api = WorkspaceApi(
+            session=dbsession,
+            current_user=admin,
+            config=self.app_config,
+        )
+        workspace = workspace_api.get_one(1)
+        app_api = ApplicationApi(
+            APP_LIST
+        )
+        default_sidebar_entry = app_api.get_default_workspace_menu_entry(workspace=workspace)  # nope8
+
         self.testapp.authorization = (
             'Basic',
             (
@@ -41,49 +59,35 @@ class TestWorkspaceEndpoint(FunctionalTest):
         assert workspace['slug'] == 'business'
         assert workspace['label'] == 'Business'
         assert workspace['description'] == 'All importants documents'
-        assert len(workspace['sidebar_entries']) == 5
-
-        # TODO - G.M - 2018-08-02 - Better test for sidebar entry, make it
-        # not fixed on active application/content-file
-        sidebar_entry = workspace['sidebar_entries'][0]
-        assert sidebar_entry['slug'] == 'dashboard'
-        assert sidebar_entry['label'] == 'Dashboard'
-        assert sidebar_entry['route'] == '/#/workspaces/1/dashboard'  # nopep8
-        assert sidebar_entry['hexcolor'] == "#252525"
-        assert sidebar_entry['fa_icon'] == "signal"
-
-        sidebar_entry = workspace['sidebar_entries'][1]
-        assert sidebar_entry['slug'] == 'contents/all'
-        assert sidebar_entry['label'] == 'All Contents'
-        assert sidebar_entry['route'] == "/#/workspaces/1/contents"  # nopep8
-        assert sidebar_entry['hexcolor'] == "#fdfdfd"
-        assert sidebar_entry['fa_icon'] == "th"
-
-        sidebar_entry = workspace['sidebar_entries'][2]
-        assert sidebar_entry['slug'] == 'contents/html-document'
-        assert sidebar_entry['label'] == 'Text Documents'
-        assert sidebar_entry['route'] == '/#/workspaces/1/contents?type=html-document'  # nopep8
-        assert sidebar_entry['hexcolor'] == "#3f52e3"
-        assert sidebar_entry['fa_icon'] == "file-text-o"
-
-        sidebar_entry = workspace['sidebar_entries'][3]
-        assert sidebar_entry['slug'] == 'contents/file'
-        assert sidebar_entry['label'] == 'Files'
-        assert sidebar_entry['route'] == "/#/workspaces/1/contents?type=file"  # nopep8
-        assert sidebar_entry['hexcolor'] == "#FF9900"
-        assert sidebar_entry['fa_icon'] == "paperclip"
-
-        sidebar_entry = workspace['sidebar_entries'][4]
-        assert sidebar_entry['slug'] == 'contents/thread'
-        assert sidebar_entry['label'] == 'Threads'
-        assert sidebar_entry['route'] == "/#/workspaces/1/contents?type=thread"  # nopep8
-        assert sidebar_entry['hexcolor'] == "#ad4cf9"
-        assert sidebar_entry['fa_icon'] == "comments-o"
+
+        assert len(workspace['sidebar_entries']) == len(default_sidebar_entry)
+        for counter, sidebar_entry in enumerate(default_sidebar_entry):
+            workspace['sidebar_entries'][counter]['slug'] = sidebar_entry.slug
+            workspace['sidebar_entries'][counter]['label'] = sidebar_entry.label
+            workspace['sidebar_entries'][counter]['route'] = sidebar_entry.route
+            workspace['sidebar_entries'][counter]['hexcolor'] = sidebar_entry.hexcolor  # nopep8
+            workspace['sidebar_entries'][counter]['fa_icon'] = sidebar_entry.fa_icon  # nopep8
 
     def test_api__update_workspace__ok_200__nominal_case(self) -> None:
         """
         Test update workspace
         """
+        dbsession = get_tm_session(self.session_factory, transaction.manager)
+        admin = dbsession.query(models.User) \
+            .filter(models.User.email == 'admin@admin.admin') \
+            .one()
+
+        workspace_api = WorkspaceApi(
+            session=dbsession,
+            current_user=admin,
+            config=self.app_config,
+        )
+        workspace = workspace_api.get_one(1)
+        app_api = ApplicationApi(
+            APP_LIST
+        )
+        default_sidebar_entry = app_api.get_default_workspace_menu_entry(workspace=workspace)  # nope8
+
         self.testapp.authorization = (
             'Basic',
             (
@@ -106,7 +110,7 @@ class TestWorkspaceEndpoint(FunctionalTest):
         assert workspace['slug'] == 'business'
         assert workspace['label'] == 'Business'
         assert workspace['description'] == 'All importants documents'
-        assert len(workspace['sidebar_entries']) == 5
+        assert len(workspace['sidebar_entries']) == len(default_sidebar_entry)
 
         # modify workspace
         res = self.testapp.put_json(
@@ -120,7 +124,7 @@ class TestWorkspaceEndpoint(FunctionalTest):
         assert workspace['slug'] == 'superworkspace'
         assert workspace['label'] == 'superworkspace'
         assert workspace['description'] == 'mysuperdescription'
-        assert len(workspace['sidebar_entries']) == 5
+        assert len(workspace['sidebar_entries']) == len(default_sidebar_entry)
 
         # after
         res = self.testapp.get(
@@ -133,7 +137,7 @@ class TestWorkspaceEndpoint(FunctionalTest):
         assert workspace['slug'] == 'superworkspace'
         assert workspace['label'] == 'superworkspace'
         assert workspace['description'] == 'mysuperdescription'
-        assert len(workspace['sidebar_entries']) == 5
+        assert len(workspace['sidebar_entries']) == len(default_sidebar_entry)
 
     def test_api__update_workspace__err_400__empty_label(self) -> None:
         """

backend/tracim_backend/tests/library/test_content_api.py

@@ -14,7 +14,7 @@ from tracim_backend.exceptions import SameValueError
 from tracim_backend.lib.core.workspace import RoleApi
 # TODO - G.M - 28-03-2018 - [WorkspaceApi] Re-enable WorkspaceApi
 from tracim_backend.lib.core.workspace import WorkspaceApi
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.revision_protection import new_revision
 from tracim_backend.models.auth import User
 from tracim_backend.models.auth import Group

backend/tracim_backend/tests/models/test_content.py

@@ -15,7 +15,7 @@ from tracim_backend.models.revision_protection import new_revision
 from tracim_backend.models import User
 from tracim_backend.models.data import ActionDescription
 from tracim_backend.models.data import ContentRevisionRO
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.data import Workspace
 from tracim_backend.tests import StandardTest
 

backend/tracim_backend/tests/models/test_content_revision.py

@@ -5,7 +5,7 @@ from sqlalchemy import inspect
 
 from tracim_backend.models import ContentRevisionRO
 from tracim_backend.models import User
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.tests import DefaultTest
 from tracim_backend.tests import eq_
 

backend/tracim_backend/views/contents_api/comment_controller.py

@@ -7,7 +7,7 @@ try:  # Python 3.5+
 except ImportError:
     from http import client as HTTPStatus
 
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.extensions import hapic
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.workspace import WorkspaceApi
@@ -20,7 +20,7 @@ from tracim_backend.views.core_api.schemas import SetCommentSchema
 from tracim_backend.views.core_api.schemas import WorkspaceAndContentIdPathSchema
 from tracim_backend.views.core_api.schemas import NoContentSchema
 from tracim_backend.exceptions import EmptyCommentContentNotAllowed
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.revision_protection import new_revision
 from tracim_backend.models.data import UserRoleInWorkspace
 

backend/tracim_backend/views/contents_api/file_controller.py

@@ -12,7 +12,7 @@ try:  # Python 3.5+
 except ImportError:
     from http import client as HTTPStatus
 
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.extensions import hapic
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.views.controllers import Controller
@@ -32,8 +32,8 @@ from tracim_backend.lib.utils.authorization import require_workspace_role
 from tracim_backend.models.data import UserRoleInWorkspace
 from tracim_backend.models.context_models import ContentInContext
 from tracim_backend.models.context_models import RevisionInContext
-from tracim_backend.models.contents import CONTENT_TYPES
-from tracim_backend.models.contents import file_type
+from tracim_backend.app_models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import file_type
 from tracim_backend.models.revision_protection import new_revision
 from tracim_backend.exceptions import EmptyLabelNotAllowed
 from tracim_backend.exceptions import PageOfPreviewNotFound

backend/tracim_backend/views/contents_api/html_document_controller.py

@@ -11,7 +11,7 @@ try:  # Python 3.5+
 except ImportError:
     from http import client as HTTPStatus
 
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.extensions import hapic
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.views.controllers import Controller
@@ -26,8 +26,8 @@ from tracim_backend.lib.utils.authorization import require_workspace_role
 from tracim_backend.exceptions import EmptyLabelNotAllowed
 from tracim_backend.models.context_models import ContentInContext
 from tracim_backend.models.context_models import RevisionInContext
-from tracim_backend.models.contents import CONTENT_TYPES
-from tracim_backend.models.contents import html_documents_type
+from tracim_backend.app_models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import html_documents_type
 from tracim_backend.models.revision_protection import new_revision
 
 SWAGGER_TAG__HTML_DOCUMENT_ENDPOINTS = 'HTML documents'

backend/tracim_backend/views/contents_api/threads_controller.py

@@ -10,7 +10,7 @@ try:  # Python 3.5+
 except ImportError:
     from http import client as HTTPStatus
 
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.extensions import hapic
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.views.controllers import Controller
@@ -25,8 +25,8 @@ from tracim_backend.lib.utils.authorization import require_workspace_role
 from tracim_backend.exceptions import EmptyLabelNotAllowed
 from tracim_backend.models.context_models import ContentInContext
 from tracim_backend.models.context_models import RevisionInContext
-from tracim_backend.models.contents import CONTENT_TYPES
-from tracim_backend.models.contents import thread_type
+from tracim_backend.app_models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import thread_type
 from tracim_backend.models.revision_protection import new_revision
 
 SWAGGER_TAG__THREAD_ENDPOINTS = 'Threads'

backend/tracim_backend/views/core_api/schemas.py

@@ -7,10 +7,10 @@ from marshmallow.validate import Range
 
 from tracim_backend.lib.utils.utils import DATETIME_FORMAT
 from tracim_backend.models.auth import Profile
-from tracim_backend.models.contents import GlobalStatus
-from tracim_backend.models.contents import CONTENT_STATUS
-from tracim_backend.models.contents import CONTENT_TYPES
-from tracim_backend.models.contents import open_status
+from tracim_backend.app_models.contents import GlobalStatus
+from tracim_backend.app_models.contents import CONTENT_STATUS
+from tracim_backend.app_models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import open_status
 from tracim_backend.models.context_models import ActiveContentFilter
 from tracim_backend.models.context_models import AutocompleteQuery
 from tracim_backend.models.context_models import ContentIdsQuery
@@ -39,6 +39,7 @@ from tracim_backend.models.context_models import ContentFilter
 from tracim_backend.models.context_models import LoginCredentials
 from tracim_backend.models.data import UserRoleInWorkspace
 from tracim_backend.models.data import ActionDescription
+from tracim_backend.app_models.validator import ALL_CONTENT_TYPES_VALIDATOR
 
 
 class UserDigestSchema(marshmallow.Schema):
@@ -357,7 +358,7 @@ class FilterContentQuerySchema(marshmallow.Schema):
     content_type = marshmallow.fields.String(
         example=CONTENT_TYPES.Any_SLUG,
         default=CONTENT_TYPES.Any_SLUG,
-        validate=OneOf(CONTENT_TYPES.endpoint_allowed_types_slug())
+        validate=ALL_CONTENT_TYPES_VALIDATOR
     )
 
     @post_load
@@ -607,7 +608,7 @@ class StatusSchema(marshmallow.Schema):
 class ContentTypeSchema(marshmallow.Schema):
     slug = marshmallow.fields.String(
         example='pagehtml',
-        validate=OneOf(CONTENT_TYPES.endpoint_allowed_types_slug()),
+        validate=ALL_CONTENT_TYPES_VALIDATOR,
     )
     fa_icon = marshmallow.fields.String(
         example='fa-file-text-o',
@@ -661,7 +662,7 @@ class ContentCreationSchema(marshmallow.Schema):
     )
     content_type = marshmallow.fields.String(
         example='html-document',
-        validate=OneOf(CONTENT_TYPES.endpoint_allowed_types_slug()),
+        validate=ALL_CONTENT_TYPES_VALIDATOR,
     )
     parent_id = marshmallow.fields.Integer(
         example=35,
@@ -696,12 +697,12 @@ class ContentDigestSchema(marshmallow.Schema):
     label = marshmallow.fields.Str(example='Intervention Report 12')
     content_type = marshmallow.fields.Str(
         example='html-document',
-        validate=OneOf(CONTENT_TYPES.endpoint_allowed_types_slug()),
+        validate=ALL_CONTENT_TYPES_VALIDATOR,
     )
     sub_content_types = marshmallow.fields.List(
         marshmallow.fields.String(
             example='html-content',
-            validate=OneOf(CONTENT_TYPES.endpoint_allowed_types_slug())
+            validate=ALL_CONTENT_TYPES_VALIDATOR
         ),
         description='list of content types allowed as sub contents. '
                     'This field is required for folder contents, '

backend/tracim_backend/views/core_api/session_controller.py

@@ -5,7 +5,7 @@ try:  # Python 3.5+
 except ImportError:
     from http import client as HTTPStatus
 
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.extensions import hapic
 from tracim_backend.lib.core.user import UserApi
 from tracim_backend.views.controllers import Controller

backend/tracim_backend/views/core_api/system_controller.py

@@ -2,18 +2,18 @@
 from pyramid.config import Configurator
 from tracim_backend.exceptions import NotAuthenticated
 from tracim_backend.exceptions import InsufficientUserProfile
+from tracim_backend.lib.core.application import ApplicationApi
 from tracim_backend.lib.utils.authorization import require_profile
 from tracim_backend.models import Group
-from tracim_backend.models.applications import applications
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 
 try:  # Python 3.5+
     from http import HTTPStatus
 except ImportError:
     from http import client as HTTPStatus
 
-from tracim_backend import TracimRequest
-from tracim_backend.extensions import hapic
+from tracim_backend.lib.utils.request import TracimRequest
+from tracim_backend.extensions import hapic, APP_LIST
 from tracim_backend.views.controllers import Controller
 from tracim_backend.views.core_api.schemas import ApplicationSchema
 from tracim_backend.views.core_api.schemas import ContentTypeSchema
@@ -30,7 +30,11 @@ class SystemController(Controller):
         """
         Get list of alls applications installed in this tracim instance.
         """
-        return applications
+        app_config = request.registry.settings['CFG']
+        app_api = ApplicationApi(
+            app_list=APP_LIST,
+        )
+        return app_api.get_all()
 
     @hapic.with_api_doc(tags=[SWAGGER_TAG_SYSTEM_ENDPOINTS])
     @require_profile(Group.TIM_USER)

backend/tracim_backend/views/core_api/user_controller.py

@@ -6,7 +6,7 @@ except ImportError:
     from http import client as HTTPStatus
 
 from tracim_backend import hapic
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.models import Group
 from tracim_backend.lib.core.group import GroupApi
 from tracim_backend.lib.core.user import UserApi
@@ -34,7 +34,7 @@ from tracim_backend.views.core_api.schemas import UserWorkspaceAndContentIdPathS
 from tracim_backend.views.core_api.schemas import ContentDigestSchema
 from tracim_backend.views.core_api.schemas import ActiveContentFilterQuerySchema
 from tracim_backend.views.core_api.schemas import WorkspaceDigestSchema
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 
 SWAGGER_TAG__USER_ENDPOINTS = 'Users'
 

backend/tracim_backend/views/core_api/workspace_controller.py

@@ -11,7 +11,7 @@ except ImportError:
     from http import client as HTTPStatus
 
 from tracim_backend import hapic
-from tracim_backend import TracimRequest
+from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.lib.core.workspace import WorkspaceApi
 from tracim_backend.lib.core.content import ContentApi
 from tracim_backend.lib.core.userworkspace import RoleApi
@@ -47,7 +47,7 @@ from tracim_backend.views.core_api.schemas import ContentDigestSchema
 from tracim_backend.views.core_api.schemas import WorkspaceSchema
 from tracim_backend.views.core_api.schemas import WorkspaceIdPathSchema
 from tracim_backend.views.core_api.schemas import WorkspaceMemberSchema
-from tracim_backend.models.contents import CONTENT_TYPES
+from tracim_backend.app_models.contents import CONTENT_TYPES
 from tracim_backend.models.revision_protection import new_revision
 
 SWAGGER_TAG_WORKSPACE_ENDPOINTS = 'Workspaces'

backend/tracim_backend/views/frontend.py

@@ -2,8 +2,9 @@ import os
 
 from pyramid.renderers import render_to_response
 from pyramid.config import Configurator
+from tracim_backend.extensions import APP_LIST
 from tracim_backend.exceptions import PageNotFound
-from tracim_backend.models.applications import applications
+from tracim_backend.lib.core.application import ApplicationApi
 from tracim_backend.views import BASE_API_V2
 from tracim_backend.lib.utils.request import TracimRequest
 from tracim_backend.views.controllers import Controller
@@ -34,6 +35,10 @@ class FrontendController(Controller):
         app_config = request.registry.settings['CFG']
         # TODO - G.M - 2018-08-07 - Refactor autogen valid app list for frontend
         frontend_apps = []
+        app_api = ApplicationApi(
+            app_list=APP_LIST,
+        )
+        applications = app_api.get_all()
         for app in applications:
             app_frontend_path = APP_FRONTEND_PATH.replace('{minislug}',
                                                           app.minislug)  # nopep8
@@ -41,11 +46,12 @@ class FrontendController(Controller):
                                     app_frontend_path)  # nopep8
             if os.path.exists(app_path):
                 frontend_apps.append(app)
+
         return render_to_response(
             self._get_index_file_path(),
             {
                 'colors': {
-                    'primary': spectra.html('#7d4e24'),
+                    'primary': spectra.html(app_config.APPS_COLORS['primary']),
                 },
                 'applications': frontend_apps,
             }