From 93d7cb50c38369dabf19b5ddf12ac0e63465c818 Mon Sep 17 00:00:00 2001
From: Antoine Nguyen <tonio@ngyn.org>
Date: Fri, 23 Aug 2013 11:39:03 +0200
Subject: [PATCH] wip: big refactoring

old userprefs application removed.
---
 modoboa/core/models.py                        |   4 -
 .../static/core}/css/userprefs.css            |   0
 modoboa/core/templates/core/user_index.html   |   2 +-
 modoboa/core/templatetags/core_tags.py        |   6 -
 modoboa/core/views/auth.py                    |   1 +
 modoboa/demo/__init__.py                      |  34 +++--
 modoboa/extensions/admin/__init__.py          |  22 ++-
 modoboa/extensions/admin/exceptions.py        |   4 +
 modoboa/extensions/admin/forms.py             |  39 ++++-
 .../admin/templates/admin}/forward.html       |   2 +-
 modoboa/extensions/admin/views/user.py        |  68 +++++++++
 .../amavis/management/commands/amnotify.py    |   4 +-
 modoboa/extensions/amavis/views.py            |   2 +-
 modoboa/extensions/sievefilters/__init__.py   |   4 +-
 modoboa/extensions/webmail/__init__.py        |   2 +-
 modoboa/extensions/webmail/app_settings.py    |   2 +-
 modoboa/static/js/twocols_nav.js              |   2 +-
 modoboa/userprefs/__init__.py                 |  20 ---
 modoboa/userprefs/app_settings.py             |  28 ----
 modoboa/userprefs/forms.py                    |  85 -----------
 .../userprefs/templates/userprefs/index.html  |  21 ---
 .../templates/userprefs/preferences.html      |   9 --
 .../templates/userprefs/profile.html          |  10 --
 modoboa/userprefs/templatetags/__init__.py    |   0
 .../userprefs/templatetags/uprefs_extras.py   |  70 ---------
 modoboa/userprefs/urls.py                     |  14 --
 modoboa/userprefs/views.py                    | 135 ------------------
 27 files changed, 159 insertions(+), 431 deletions(-)
 rename modoboa/{userprefs/static/userprefs => core/static/core}/css/userprefs.css (100%)
 rename modoboa/{userprefs/templates/userprefs => extensions/admin/templates/admin}/forward.html (90%)
 create mode 100644 modoboa/extensions/admin/views/user.py
 delete mode 100644 modoboa/userprefs/__init__.py
 delete mode 100644 modoboa/userprefs/app_settings.py
 delete mode 100644 modoboa/userprefs/forms.py
 delete mode 100644 modoboa/userprefs/templates/userprefs/index.html
 delete mode 100644 modoboa/userprefs/templates/userprefs/preferences.html
 delete mode 100644 modoboa/userprefs/templates/userprefs/profile.html
 delete mode 100644 modoboa/userprefs/templatetags/__init__.py
 delete mode 100644 modoboa/userprefs/templatetags/uprefs_extras.py
 delete mode 100644 modoboa/userprefs/urls.py
 delete mode 100644 modoboa/userprefs/views.py

diff --git a/modoboa/core/models.py b/modoboa/core/models.py
index d4e4b277d..2c90a4604 100644
--- a/modoboa/core/models.py
+++ b/modoboa/core/models.py
@@ -181,10 +181,6 @@ class User(AbstractBaseUser, PermissionsMixin):
         return [{"name": "account", "label": _("account"), "type": "idt"},
                 {"name": self.group, "label": self.group, "type": "grp", "color": "info"}]
 
-    # @property
-    # def has_mailbox(self):
-    #     return self.mailbox_set.count() != 0
-
     @property
     def fullname(self):
         if self.first_name != u"":
diff --git a/modoboa/userprefs/static/userprefs/css/userprefs.css b/modoboa/core/static/core/css/userprefs.css
similarity index 100%
rename from modoboa/userprefs/static/userprefs/css/userprefs.css
rename to modoboa/core/static/core/css/userprefs.css
diff --git a/modoboa/core/templates/core/user_index.html b/modoboa/core/templates/core/user_index.html
index e721beabe..993dc9163 100644
--- a/modoboa/core/templates/core/user_index.html
+++ b/modoboa/core/templates/core/user_index.html
@@ -1,7 +1,7 @@
 {% extends "twocols.html" %}{% load i18n core_tags libextras %}
 {% block pagetitle %}{% trans "Settings" %}{% endblock %}
 {% block html_head %}
-<link rel="stylesheet" type="text/css" href="{{ STATIC_URL }}userprefs/css/userprefs.css" />
+<link rel="stylesheet" type="text/css" href="{{ STATIC_URL }}core/css/userprefs.css" />
 <script type="text/javascript" src="{{ STATIC_URL }}js/history.js"></script>
 <script type="text/javascript" src="{{ STATIC_URL }}js/listing.js"></script>
 <script type="text/javascript" src="{{ STATIC_URL }}js/twocols_nav.js"></script>
diff --git a/modoboa/core/templatetags/core_tags.py b/modoboa/core/templatetags/core_tags.py
index 1c7933ed9..53ff25b7d 100644
--- a/modoboa/core/templatetags/core_tags.py
+++ b/modoboa/core/templatetags/core_tags.py
@@ -135,12 +135,6 @@ def uprefs_menu(selection, user):
          "url": "preferences/",
          "label": _("Preferences")},
     ]
-    # if user.has_mailbox:
-    #     entries.insert(0, {"name": "forward",
-    #                        "class": "ajaxlink",
-    #                        "url": "forward/",
-    #                        "label": _("Forward")})
-
     entries += events.raiseQueryEvent("UserMenuDisplay", "uprefs_menu", user)
     entries = sorted(entries, key=lambda e: e["label"])
     return render_to_string('common/menu.html', {
diff --git a/modoboa/core/views/auth.py b/modoboa/core/views/auth.py
index 3e2b25561..cb84e6c36 100644
--- a/modoboa/core/views/auth.py
+++ b/modoboa/core/views/auth.py
@@ -23,6 +23,7 @@ def dologin(request):
                 login(request, user)
                 if not form.cleaned_data["rememberme"]:
                     request.session.set_expiry(0)
+                events.raiseEvent("UserLoggedIn", request, form)
                 # if request.user.has_mailbox:
                 #     request.session["password"] = encrypt(form.cleaned_data["password"])
 
diff --git a/modoboa/demo/__init__.py b/modoboa/demo/__init__.py
index 739076d53..69ccfecc5 100755
--- a/modoboa/demo/__init__.py
+++ b/modoboa/demo/__init__.py
@@ -6,27 +6,29 @@ from django.template.loader import render_to_string
 from modoboa.lib import events
 from modoboa.lib.webutils import static_url
 
+
 @events.observe("UserMenuDisplay")
 def menu(target, user):
     if target != "top_menu_middle":
         return []
-    if not user.has_mailbox:
+    if not user.mailbox_set.count():
         return []
     return [
-        {"name"  : "demo",
-         "label" : _("Test messages"),
-         "img" : static_url("pics/demo.png"),
-         "class" : "topdropdown",
-         "menu"  : [
-                {"name"  : "sendvirus",
-                 "label" :  _("Send virus"),
-                 "url"   : reverse("modoboa.demo.views.send_virus")},
-                {"name"  : "sendspam",
-                 "label" :  _("Send spam"),
-                 "url"   : reverse("modoboa.demo.views.send_spam")}
-                ]
+        {"name": "demo",
+         "label": _("Test messages"),
+         "img": static_url("pics/demo.png"),
+         "class": "topdropdown",
+         "menu": [
+             {"name": "sendvirus",
+              "label":  _("Send virus"),
+              "url": reverse("modoboa.demo.views.send_virus")},
+             {"name": "sendspam",
+              "label":  _("Send spam"),
+              "url": reverse("modoboa.demo.views.send_spam")}
+         ]
          }
-        ]
+    ]
+
 
 @events.observe("GetAnnouncement")
 def announcement(target):
@@ -35,13 +37,15 @@ def announcement(target):
         return [txt]
     return ""
 
+
 @events.observe("PasswordChange")
 def password_change(user):
     return [user.id == 1]
 
+
 @events.observe("GetStaticContent")
 def get_static_content(user):
-    if not user.has_mailbox:
+    if not user.mailbox_set.count():
         return []
     return """<script type="text/javascript">
 $(document).ready(function() {
diff --git a/modoboa/extensions/admin/__init__.py b/modoboa/extensions/admin/__init__.py
index b966c4d75..793ea13ae 100644
--- a/modoboa/extensions/admin/__init__.py
+++ b/modoboa/extensions/admin/__init__.py
@@ -53,8 +53,14 @@ class AdminConsole(ModoExtension):
 exts_pool.register_extension(AdminConsole, show=False)
 
 
+@events.observe("ExtraUprefsRoutes")
+def extra_routes():
+    return [(r'^user/forward/',
+             'modoboa.extensions.admin.views.user.forward'), ]
+
+
 @events.observe("AdminMenuDisplay")
-def menu(target, user):
+def admin_menu(target, user):
     if target != "top_menu":
         return []
     entries = []
@@ -73,6 +79,20 @@ def menu(target, user):
     return entries
 
 
+@events.observe("UserMenuDisplay")
+def user_menu(target, user):
+    if target != "uprefs_menu":
+        return []
+    if not user.mailbox_set.count():
+        return []
+    return [
+        {"name": "forward",
+         "class": "ajaxlink",
+         "url": "forward/",
+         "label": ugettext_lazy("Forward")}
+    ]
+
+
 @events.observe("RoleChanged")
 def grant_access_to_all_objects(user, role):
     from django.contrib.contenttypes.models import ContentType
diff --git a/modoboa/extensions/admin/exceptions.py b/modoboa/extensions/admin/exceptions.py
index 9ee8ced0d..d0414b27b 100644
--- a/modoboa/extensions/admin/exceptions.py
+++ b/modoboa/extensions/admin/exceptions.py
@@ -13,3 +13,7 @@ class AdminError(ModoboaException):
     application.
     """
     pass
+
+
+class BadDestination(Exception):
+    pass
diff --git a/modoboa/extensions/admin/forms.py b/modoboa/extensions/admin/forms.py
index 90f27dbe0..e53f1ce90 100644
--- a/modoboa/extensions/admin/forms.py
+++ b/modoboa/extensions/admin/forms.py
@@ -6,12 +6,15 @@ from django.http import QueryDict
 from modoboa.lib import events, parameters
 from modoboa.lib.emailutils import split_mailbox
 from modoboa.lib.permissions import get_account_roles
-from modoboa.lib.formutils import *
+from modoboa.lib.formutils import (
+    DomainNameField, YesNoField, DynamicForm, TabForms
+)
+from modoboa.lib.exceptions import PermDeniedException
 from modoboa.core.models import User
 from modoboa.extensions.admin.templatetags.admin_tags import gender
-from modoboa.extensions.admin.exceptions import AdminError
+from modoboa.extensions.admin.exceptions import AdminError, BadDestination
 from modoboa.extensions.admin.models import (
-    Domain, DomainAlias, Mailbox, Alias
+    Domain, DomainAlias, Mailbox, Alias, Quota
 )
 
 
@@ -670,3 +673,33 @@ class AccountForm(TabForms):
             return
         for f in self.forms[2:]:
             f["instance"].save()
+
+
+class ForwardForm(forms.Form):
+    dest = forms.CharField(
+        label=ugettext_lazy("Recipient(s)"),
+        widget=forms.Textarea,
+        required=False,
+        help_text=ugettext_lazy("Indicate one or more recipients separated by a ','")
+    )
+    keepcopies = forms.BooleanField(
+        label=ugettext_lazy("Keep local copies"),
+        required=False,
+        help_text=ugettext_lazy("Forward messages and store copies into your local mailbox")
+    )
+
+    def parse_dest(self):
+        self.dests = []
+        rawdata = self.cleaned_data["dest"].strip()
+        if rawdata == "":
+            return
+        for d in rawdata.split(","):
+            local_part, domname = split_mailbox(d)
+            if not local_part or not domname or not len(domname):
+                raise BadDestination("Invalid mailbox syntax for %s" % d)
+            try:
+                mb = Domain.objects.get(name=domname)
+            except Domain.DoesNotExist:
+                self.dests += [d]
+            else:
+                raise BadDestination(_("You can't define a forward to a local destination. Please ask your administrator to create an alias instead."))
diff --git a/modoboa/userprefs/templates/userprefs/forward.html b/modoboa/extensions/admin/templates/admin/forward.html
similarity index 90%
rename from modoboa/userprefs/templates/userprefs/forward.html
rename to modoboa/extensions/admin/templates/admin/forward.html
index b0b607e14..35e332ed0 100644
--- a/modoboa/userprefs/templates/userprefs/forward.html
+++ b/modoboa/extensions/admin/templates/admin/forward.html
@@ -1,7 +1,7 @@
 {% load i18n libextras %}{% load url from future %}
 <h2>{% trans "Forward" %} <small>{% trans "Automatically forward emails to another address" %}</small></h2>
 <hr>
-<form class="form-horizontal" method="POST" action="{% url 'modoboa.userprefs.views.forward' %}" 
+<form class="form-horizontal" method="POST" action="{% url 'modoboa.extensions.admin.views.user.forward' %}" 
       id="uprefs_form">{% csrf_token %}
   {% render_form form %}
   <div class="form-actions">
diff --git a/modoboa/extensions/admin/views/user.py b/modoboa/extensions/admin/views/user.py
new file mode 100644
index 000000000..ad799dc17
--- /dev/null
+++ b/modoboa/extensions/admin/views/user.py
@@ -0,0 +1,68 @@
+from django.contrib.auth.decorators import login_required
+from django.utils.translation import ugettext as _
+from modoboa.lib.exceptions import ModoboaException
+from modoboa.lib.webutils import (
+    ajax_response, ajax_simple_response, _render_to_string
+)
+from modoboa.extensions.admin.lib import needs_mailbox
+from modoboa.extensions.admin.exceptions import BadDestination
+from modoboa.extensions.admin.models import Alias, Mailbox
+from modoboa.extensions.admin.forms import ForwardForm
+
+
+@login_required
+@needs_mailbox()
+def forward(request, tplname='admin/forward.html'):
+    try:
+        mb = request.user.mailbox_set.all()[0]
+    except IndexError:
+        raise ModoboaException(
+            _("You need a mailbox in order to define a forward")
+        )
+    try:
+        al = Alias.objects.get(address=mb.address,
+                               domain__name=mb.domain.name)
+    except Alias.DoesNotExist:
+        al = None
+    if request.method == "POST":
+        form = ForwardForm(request.POST)
+        error = None
+        if form.is_valid():
+            try:
+                if al is None:
+                    al = Alias()
+                    al.address = mb.address
+                    al.domain = mb.domain
+                    al.enabled = mb.user.is_active
+                intdests = []
+                if form.cleaned_data["keepcopies"]:
+                    intdests += [mb]
+                form.parse_dest()
+                al.save(intdests, form.dests)
+                if request.user.group != "SimpleUsers":
+                    al.post_create(request.user)
+                return ajax_response(request, respmsg=_("Forward updated"))
+            except BadDestination, e:
+                error = str(e)
+
+        return ajax_simple_response(dict(
+            status="ko",
+            errors=form.errors,
+            respmsg=error
+        ))
+
+    form = ForwardForm()
+    if al is not None:
+        form.fields["dest"].initial = al.extmboxes
+        try:
+            selfmb = al.mboxes.get(pk=mb.id)
+        except Mailbox.DoesNotExist:
+            pass
+        else:
+            form.fields["keepcopies"].initial = True
+    return ajax_simple_response({
+        "status": "ok",
+        "content": _render_to_string(request, tplname, {
+            "form": form
+        })
+    })
diff --git a/modoboa/extensions/amavis/management/commands/amnotify.py b/modoboa/extensions/amavis/management/commands/amnotify.py
index ec3df5ea8..083cce65f 100644
--- a/modoboa/extensions/amavis/management/commands/amnotify.py
+++ b/modoboa/extensions/amavis/management/commands/amnotify.py
@@ -71,7 +71,7 @@ class Command(BaseCommand):
         self.listingurl = self.baseurl + reverse("modoboa.extensions.amavis.views._listing") + "?viewrequests=1"
 
         for da in User.objects.filter(groups__name="DomainAdmins"):
-            if not da.has_mailbox:
+            if not da.mailbox_set.count():
                 continue
             rcpt = da.mailbox_set.all()[0].full_address
             reqs = get_wrapper().get_domains_pending_requests(da.get_domains())
@@ -84,7 +84,7 @@ class Command(BaseCommand):
                 print "No release request currently pending"
             return
         for su in User.objects.filter(is_superuser=True):
-            if not su.has_mailbox:
+            if not su.mailbox_set.count():
                 continue
             rcpt = su.mailbox_set.all()[0].full_address
             self.send_pr_notification(rcpt, reqs)
diff --git a/modoboa/extensions/amavis/views.py b/modoboa/extensions/amavis/views.py
index 255018a53..52be6b4bd 100644
--- a/modoboa/extensions/amavis/views.py
+++ b/modoboa/extensions/amavis/views.py
@@ -155,7 +155,7 @@ def viewmail(request, mail_id):
     else:
         rcpt = None
 
-    if request.user.has_mailbox:
+    if request.user.mailbox_set.count():
         mb = Mailbox.objects.get(user=request.user)
         if not rcpt or rcpt == mb.full_address:
             msgrcpt = get_wrapper().get_recipient_message(mb.full_address, mail_id)
diff --git a/modoboa/extensions/sievefilters/__init__.py b/modoboa/extensions/sievefilters/__init__.py
index 34fea4f6e..c5b88c95f 100644
--- a/modoboa/extensions/sievefilters/__init__.py
+++ b/modoboa/extensions/sievefilters/__init__.py
@@ -33,7 +33,7 @@ def menu(target, user):
     return [
         {"name": "sievefilters",
          "label": _("Message filters"),
-         "url": reverse("modobooa.extensions.sievefilters.views.index"),
+         "url": reverse("modoboa.extensions.sievefilters.views.index"),
          "img": "icon-check"}
     ]
 
@@ -42,7 +42,7 @@ def menu(target, user):
 def userlogout(request):
     from .lib import SieveClient
 
-    if not request.user.has_mailbox:
+    if not request.user.mailbox_set.count():
         return
     try:
         sc = SieveClient(user=request.user.username,
diff --git a/modoboa/extensions/webmail/__init__.py b/modoboa/extensions/webmail/__init__.py
index c25468e2a..8b336b8b1 100644
--- a/modoboa/extensions/webmail/__init__.py
+++ b/modoboa/extensions/webmail/__init__.py
@@ -43,7 +43,7 @@ def userlogout(request):
     from .lib import IMAPconnector
     from .exceptions import ImapError
 
-    if not request.user.has_mailbox:
+    if not request.user.mailbox_set.count():
         return
     try:
         m = IMAPconnector(user=request.user.username,
diff --git a/modoboa/extensions/webmail/app_settings.py b/modoboa/extensions/webmail/app_settings.py
index 119d7b804..71a2040d5 100644
--- a/modoboa/extensions/webmail/app_settings.py
+++ b/modoboa/extensions/webmail/app_settings.py
@@ -142,4 +142,4 @@ class UserSettings(UserParametersForm):
 
     @staticmethod
     def has_access(user):
-        return user.has_mailbox
+        return user.mailbox_set.count() != 0
diff --git a/modoboa/static/js/twocols_nav.js b/modoboa/static/js/twocols_nav.js
index fcdc2477a..d8d3ed337 100644
--- a/modoboa/static/js/twocols_nav.js
+++ b/modoboa/static/js/twocols_nav.js
@@ -32,7 +32,7 @@ TwocolsNav.prototype = {
     },
 
     display_errors: function(data) {
-        if (data.errors == undefined) {
+        if (data.errors === undefined || !data.errors.length) {
             if (data.respmsg) {
                 $("body").notify("error", data.respmsg);
             }
diff --git a/modoboa/userprefs/__init__.py b/modoboa/userprefs/__init__.py
deleted file mode 100644
index d542429da..000000000
--- a/modoboa/userprefs/__init__.py
+++ /dev/null
@@ -1,20 +0,0 @@
-# coding: utf-8
-from django.utils.translation import ugettext_lazy
-from modoboa.extensions import ModoExtension, exts_pool
-from modoboa.lib import parameters
-
-class UserPreferences(ModoExtension):
-    name = "userprefs"
-    label = ugettext_lazy("User preferences")
-    version = "1.0"
-    description = ugettext_lazy("Per-user settings")
-    always_active = True
-
-    def load(self):
-        from app_settings import UserSettings
-        parameters.register(UserSettings, ugettext_lazy("General"))
-
-    def destroy(self):
-        parameters.unregister()
-
-exts_pool.register_extension(UserPreferences, show=False)
diff --git a/modoboa/userprefs/app_settings.py b/modoboa/userprefs/app_settings.py
deleted file mode 100644
index 7652fada0..000000000
--- a/modoboa/userprefs/app_settings.py
+++ /dev/null
@@ -1,28 +0,0 @@
-# coding: utf-8
-from django import forms
-from django.utils.translation import ugettext_lazy
-from django.conf import settings
-from modoboa.lib.parameters import UserParametersForm
-from modoboa.lib.formutils import SeparatorField
-
-
-def translate_language_code(value):
-    if "-" in value:
-        return value.split("-")[0]
-    return value
-
-
-class UserSettings(UserParametersForm):
-    app = "general"
-
-    sep = SeparatorField(label=ugettext_lazy("Display"))
-
-    lang = forms.ChoiceField(
-        initial=translate_language_code(settings.LANGUAGE_CODE),
-        label=ugettext_lazy("Prefered language"),
-        choices=[("de", "deutsch"), ("en", "english"),
-                ("es", "español"), ("fr", "français"),
-                ("it", "italiano"), ("pt", "português"),
-                ("sv", "svenska"),],
-        help_text=ugettext_lazy("Prefered language to display pages")
-    )
diff --git a/modoboa/userprefs/forms.py b/modoboa/userprefs/forms.py
deleted file mode 100644
index 88200bc9c..000000000
--- a/modoboa/userprefs/forms.py
+++ /dev/null
@@ -1,85 +0,0 @@
-# coding: utf-8
-from django import forms
-from django.utils.translation import ugettext as _, ugettext_lazy
-
-from modoboa.admin.models import User, Domain
-from modoboa.lib.emailutils import split_mailbox
-from modoboa.lib import parameters
-
-
-class BadDestination(Exception):
-    pass
-
-class ProfileForm(forms.ModelForm):
-    oldpassword = forms.CharField(label=ugettext_lazy("Old password"), required=False,
-                                  widget=forms.PasswordInput)
-    newpassword = forms.CharField(label=ugettext_lazy("New password"), required=False,
-                                  widget=forms.PasswordInput)
-    confirmation = forms.CharField(label=ugettext_lazy("Confirmation"), required=False,
-                                   widget=forms.PasswordInput)
-
-    class Meta:
-        model = User
-        fields = ("first_name", "last_name")
-
-    def __init__(self, update_password, *args, **kwargs):
-        super(ProfileForm, self).__init__(*args, **kwargs)
-        if not update_password:
-            del self.fields["oldpassword"]
-            del self.fields["newpassword"]
-            del self.fields["confirmation"]
-
-    def clean_oldpassword(self):
-        if self.cleaned_data["oldpassword"] == "":
-            return self.cleaned_data["oldpassword"]
-
-        if parameters.get_admin("AUTHENTICATION_TYPE", app="admin") != "local":
-            return self.cleaned_data["oldpassword"]
-
-        if not self.instance.check_password(self.cleaned_data["oldpassword"]):
-            raise forms.ValidationError(_("Old password mismatchs"))
-        return self.cleaned_data["oldpassword"]
-
-    def clean_confirmation(self):
-        if self.cleaned_data["newpassword"] != self.cleaned_data["confirmation"]:
-            raise forms.ValidationError(_("Passwords mismatch"))
-        return self.cleaned_data["confirmation"]
-
-    def save(self, commit=True):
-        user = super(ProfileForm, self).save(commit=False)
-        if commit:
-            if self.cleaned_data.has_key("confirmation") and \
-                    self.cleaned_data["confirmation"] != "":
-                user.set_password(self.cleaned_data["confirmation"], self.cleaned_data["oldpassword"])
-            user.save()
-        return user
-
-class ForwardForm(forms.Form):
-    dest = forms.CharField(
-        label=ugettext_lazy("Recipient(s)"),
-        widget=forms.Textarea,
-        required=False,
-        help_text=ugettext_lazy("Indicate one or more recipients separated by a ','")
-        )
-    keepcopies = forms.BooleanField(
-        label=ugettext_lazy("Keep local copies"),
-        required=False,
-        help_text=ugettext_lazy("Forward messages and store copies into your local mailbox")
-        )
-
-    def parse_dest(self):
-        self.dests = []
-        rawdata = self.cleaned_data["dest"].strip()
-        if rawdata == "":
-            return
-        for d in rawdata.split(","):
-            local_part, domname = split_mailbox(d)
-            if not local_part or not domname or not len(domname):
-                raise BadDestination("Invalid mailbox syntax for %s" % d)
-            try:
-                mb = Domain.objects.get(name=domname)
-            except Domain.DoesNotExist:
-                self.dests += [d]
-            else:
-                raise BadDestination(_("You can't define a forward to a local destination. Please ask your administrator to create an alias instead."))
-
diff --git a/modoboa/userprefs/templates/userprefs/index.html b/modoboa/userprefs/templates/userprefs/index.html
deleted file mode 100644
index 6a240e0c9..000000000
--- a/modoboa/userprefs/templates/userprefs/index.html
+++ /dev/null
@@ -1,21 +0,0 @@
-{% extends "twocols.html" %}{% load i18n uprefs_extras libextras %}
-{% block pagetitle %}{% trans "Settings" %}{% endblock %}
-{% block html_head %}
-<link rel="stylesheet" type="text/css" href="{{ STATIC_URL }}userprefs/css/userprefs.css" />
-<script type="text/javascript" src="{{ STATIC_URL }}js/history.js"></script>
-<script type="text/javascript" src="{{ STATIC_URL }}js/listing.js"></script>
-<script type="text/javascript" src="{{ STATIC_URL }}js/twocols_nav.js"></script>
-<script type="text/javascript">
-{{ extrajs|safe }}
-$(document).ready(function() {
-    var userprefs = new TwocolsNav({
-        deflocation: "profile/",
-        formid: "uprefs_form",
-        divid: "uprefs_content",
-        reload_exceptions: ["profile"]
-    });
-});
-</script>
-{% endblock %}
-{% block leftcol %}<div class="well sidebar-nav">{% uprefs_menu left_selection user %}</div>{% endblock %}
-{% block apparea %}<div id="uprefs_content"></div>{% endblock %}
diff --git a/modoboa/userprefs/templates/userprefs/preferences.html b/modoboa/userprefs/templates/userprefs/preferences.html
deleted file mode 100644
index 26a514dd7..000000000
--- a/modoboa/userprefs/templates/userprefs/preferences.html
+++ /dev/null
@@ -1,9 +0,0 @@
-{% load i18n %}{% load url from future %}
-<h2>{% trans "Preferences" %} <small>{% trans "Customize Modoboa" %}</small></h2>
-<hr>
-<form class="form-horizontal" method="POST" action="{% url 'modoboa.userprefs.views.preferences' %}" 
-      id="uprefs_form">{% csrf_token %}
-  {% include "common/parameters.html" %}
-  <hr>
-  <button type="submit" class="btn btn-primary" id="update" name="update">{% trans 'Save' %}</button>
-</form>
diff --git a/modoboa/userprefs/templates/userprefs/profile.html b/modoboa/userprefs/templates/userprefs/profile.html
deleted file mode 100644
index f5df0b64e..000000000
--- a/modoboa/userprefs/templates/userprefs/profile.html
+++ /dev/null
@@ -1,10 +0,0 @@
-{% load i18n libextras %}{% load url from future %}
-<h2>{% trans "Profile" %} <small>{% trans "Update your personal information" %}</small></h2>
-<hr>
-<form class="form-horizontal" method="POST" action="{% url 'modoboa.userprefs.views.profile' %}" 
-      id="uprefs_form">{% csrf_token %}
-  {% render_form form %}
-  <div class="form-actions">
-    <button type="submit" class="btn btn-primary" id="update" name="update">{% trans 'Update' %}</button>
-  </div>
-</form>
diff --git a/modoboa/userprefs/templatetags/__init__.py b/modoboa/userprefs/templatetags/__init__.py
deleted file mode 100644
index e69de29bb..000000000
diff --git a/modoboa/userprefs/templatetags/uprefs_extras.py b/modoboa/userprefs/templatetags/uprefs_extras.py
deleted file mode 100644
index 8a73ad118..000000000
--- a/modoboa/userprefs/templatetags/uprefs_extras.py
+++ /dev/null
@@ -1,70 +0,0 @@
-# coding: utf-8
-from django import template
-from django.template.loader import render_to_string
-from django.utils.translation import ugettext as _
-from django.core.urlresolvers import reverse
-from modoboa.lib import events
-
-register = template.Library()
-
-
-@register.simple_tag
-def uprefs_menu(selection, user):
-    entries = [
-        {"name": "profile",
-         "class": "ajaxlink",
-         "url": "profile/",
-         "label": _("Profile")},
-        {"name": "preferences",
-         "class": "ajaxlink",
-         "url": "preferences/",
-         "label": _("Preferences")},
-    ]
-    if user.has_mailbox:
-        entries.insert(0, {"name": "forward",
-                           "class": "ajaxlink",
-                           "url": "forward/",
-                           "label": _("Forward")})
-
-    entries += events.raiseQueryEvent("UserMenuDisplay", "uprefs_menu", user)
-    entries = sorted(entries, key=lambda e: e["label"])
-    return render_to_string('common/menu.html', {
-        "entries": entries,
-        "css": "nav nav-list",
-        "selection": selection,
-        "user": user
-    })
-
-
-@register.simple_tag
-def user_menu(user, selection):
-    entries = [
-        {"name": "user",
-         "img": "icon-user icon-white",
-         "label": user.fullname,
-         "menu": [
-                {"name": "settings",
-                 "img": "icon-list",
-                 "label": _("Settings"),
-                 "url": reverse("modoboa.userprefs.views.index")}
-         ]}
-    ]
-
-    entries[0]["menu"] += \
-        events.raiseQueryEvent("UserMenuDisplay", "options_menu", user) \
-        + [{"name": "logout",
-            "url": reverse("modoboa.auth.views.dologout"),
-            "label": _("Logout"),
-            "img": "icon-off"}]
-
-    return render_to_string("common/menulist.html", {
-        "selection": selection, "entries": entries, "user": user
-    })
-
-
-@register.simple_tag
-def loadextmenu(selection, user):
-    menu = events.raiseQueryEvent("UserMenuDisplay", "top_menu", user)
-    return render_to_string('common/menulist.html', {
-        "selection": selection, "entries": menu, "user": user
-    })
diff --git a/modoboa/userprefs/urls.py b/modoboa/userprefs/urls.py
deleted file mode 100644
index 05e214541..000000000
--- a/modoboa/userprefs/urls.py
+++ /dev/null
@@ -1,14 +0,0 @@
-# coding: utf-8
-from django.conf.urls import patterns, url
-from modoboa.lib import events
-
-urlpatterns = patterns('modoboa.userprefs.views',
-    url(r'^$', 'index'),
-    (r'^preferences/$', 'preferences'),
-    (r'^profile/$', 'profile'),
-    (r'^forward/$', 'forward'),
-)
-
-urlpatterns += patterns('',
-    *events.raiseQueryEvent("ExtraUprefsRoutes")
-)
diff --git a/modoboa/userprefs/views.py b/modoboa/userprefs/views.py
deleted file mode 100644
index be70d1958..000000000
--- a/modoboa/userprefs/views.py
+++ /dev/null
@@ -1,135 +0,0 @@
-# coding: utf-8
-from django.shortcuts import render
-from django.contrib.auth.decorators import login_required, user_passes_test
-from django.utils.translation import ugettext as _
-from modoboa.lib import parameters, events
-from modoboa.lib.decorators import needs_mailbox
-from modoboa.lib.exceptions import ModoboaException
-from modoboa.lib.webutils import (
-    ajax_response, ajax_simple_response, _render_to_string
-)
-from modoboa.admin.models import Mailbox, Alias
-from modoboa.auth.lib import encrypt
-from forms import ForwardForm, BadDestination, ProfileForm
-
-
-@login_required
-def index(request, tplname="userprefs/index.html"):
-    extrajs = events.raiseQueryEvent("ExtraUprefsJS", request.user)
-    return render(request, tplname, {
-        "selection": "user",
-        "extrajs": "".join(extrajs)
-    })
-
-
-@login_required
-@needs_mailbox()
-def forward(request, tplname='userprefs/forward.html'):
-    try:
-        mb = request.user.mailbox_set.all()[0]
-    except IndexError:
-        raise ModoboaException(
-            _("You need a mailbox in order to define a forward")
-        )
-    try:
-        al = Alias.objects.get(address=mb.address,
-                               domain__name=mb.domain.name)
-    except Alias.DoesNotExist:
-        al = None
-    if request.method == "POST":
-        form = ForwardForm(request.POST)
-        error = None
-        if form.is_valid():
-            try:
-                if al is None:
-                    al = Alias()
-                    al.address = mb.address
-                    al.domain = mb.domain
-                    al.enabled = mb.user.is_active
-                intdests = []
-                if form.cleaned_data["keepcopies"]:
-                    intdests += [mb]
-                form.parse_dest()
-                al.save(intdests, form.dests)
-                if request.user.group != "SimpleUsers":
-                    al.post_create(request.user)
-                return ajax_response(request, respmsg=_("Forward updated"))
-            except BadDestination, e:
-                error = str(e)
-
-        return ajax_simple_response(dict(
-            status="ko",
-            errors=form.errors,
-            respmsg=error
-        ))
-
-    form = ForwardForm()
-    if al is not None:
-        form.fields["dest"].initial = al.extmboxes
-        try:
-            selfmb = al.mboxes.get(pk=mb.id)
-        except Mailbox.DoesNotExist:
-            pass
-        else:
-            form.fields["keepcopies"].initial = True
-    return ajax_simple_response({
-        "status": "ok",
-        "content": _render_to_string(request, tplname, {
-            "form": form
-        })
-    })
-
-
-@login_required
-def profile(request, tplname='userprefs/profile.html'):
-    update_password = True
-    if True in events.raiseQueryEvent("PasswordChange", request.user):
-        update_password = False
-
-    if request.method == "POST":
-        form = ProfileForm(
-            update_password, request.POST, instance=request.user
-        )
-        if form.is_valid():
-            form.save()
-            if update_password and form.cleaned_data["confirmation"] != "":
-                request.session["password"] = encrypt(form.cleaned_data["confirmation"])
-            return ajax_simple_response(dict(
-                status="ok", respmsg=_("Profile updated")
-            ))
-        return ajax_simple_response({
-            "status": "ko",
-            "errors": form.errors
-        })
-
-    form = ProfileForm(update_password, instance=request.user)
-    return ajax_simple_response({
-        "status": "ok",
-        "content": _render_to_string(request, tplname, {
-            "form": form
-        })
-    })
-
-
-@login_required
-def preferences(request):
-    if request.method == "POST":
-        for formdef in parameters.get_user_forms(request.user, request.POST)():
-            form = formdef["form"]
-            if form.is_valid():
-                form.save()
-                continue
-            return ajax_simple_response({
-                "status": "ko", "prefix": form.app, "errors": form.errors
-            })
-
-        return ajax_simple_response({
-            "status": "ok", "respmsg": _("Preferences saved")
-        })
-
-    return ajax_simple_response({
-        "status": "ok",
-        "content": _render_to_string(request, "userprefs/preferences.html", {
-            "forms": parameters.get_user_forms(request.user)
-        })
-    })