SF.net SVN: postfixadmin:[1428] trunk

[<chr...@us...>]

Revision: 1428
          http://sourceforge.net/p/postfixadmin/code/1428
Author:   christian_boltz
Date:     2013-02-16 20:42:17 +0000 (Sat, 16 Feb 2013)
Log Message:
-----------
PFAHandler.php:
- introduce public array $infomsg[] (for success messages)
- store(): write successmessage to $this->infomsg[]

edit.php:
- display $handler->infomsg instead of hardcoded success message

AdminHandler.php, AliasHandler.php, AliasdomainHandler.php,
DomainHandler.php, MailboxHandler.php:
- move successmsg from webformConfig() to initMsg()

Modified Paths:
--------------
    trunk/edit.php
    trunk/model/AdminHandler.php
    trunk/model/AliasHandler.php
    trunk/model/AliasdomainHandler.php
    trunk/model/DomainHandler.php
    trunk/model/MailboxHandler.php
    trunk/model/PFAHandler.php

Modified: trunk/edit.php
===================================================================
--- trunk/edit.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/edit.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -139,10 +139,7 @@
         if (!$handler->store()) {
             $errormsg = $handler->errormsg;
         } else {
-            flash_info(Lang::read($formconf['successmessage']) . " (" . $values[$id_field] . ")");
-            # TODO: - use a sprintf string
-            # TODO: - get the success message from DomainHandler
-            # TODO: - use a different success message for create and edit
+            flash_info($handler->infomsg);
 
             if (count($handler->errormsg)) { # might happen if domain_postcreation fails
                 flash_error($handler->errormsg);

Modified: trunk/model/AdminHandler.php
===================================================================
--- trunk/model/AdminHandler.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/model/AdminHandler.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -86,25 +86,20 @@
         if ($this->new) {
             $this->msg['logname'] = 'create_admin';
             $this->msg['store_error'] = 'pAdminCreate_admin_result_error';
+            $this->msg['successmessage'] = 'pAdminCreate_admin_result_success';
         } else {
             $this->msg['logname'] = 'edit_admin';
             $this->msg['store_error'] = 'pAdminEdit_admin_result_error';
+            $this->msg['successmessage'] = 'pAdminEdit_admin_result_success';
         }
     }
 
     public function webformConfig() {
-        if ($this->new) {
-            $successmsg = 'pAdminCreate_admin_result_success';
-        } else {
-            $successmsg = 'pAdminEdit_admin_result_success';
-        }
-
         return array(
             # $PALANG labels
             'formtitle_create' => 'pAdminCreate_admin_welcome',
             'formtitle_edit' => 'pAdminEdit_admin_welcome',
             'create_button' => 'pAdminCreate_admin_button',
-            'successmessage' => $successmsg,
 
             # various settings
             'required_role' => 'global-admin',

Modified: trunk/model/AliasHandler.php
===================================================================
--- trunk/model/AliasHandler.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/model/AliasHandler.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -74,9 +74,11 @@
         if ($this->new) {
             $this->msg['logname'] = 'create_alias';
             $this->msg['store_error'] = 'pCreate_alias_result_error';
+            $this->msg['successmessage'] = 'pCreate_alias_result_success';
         } else {
             $this->msg['logname'] = 'edit_alias';
             $this->msg['store_error'] = 'pEdit_alias_result_error';
+            $this->msg['successmessage'] = 'pCreate_alias_result_success'; # TODO: better message for edit
         }
     }
 
@@ -93,7 +95,6 @@
             'formtitle_create'  => 'pCreate_alias_welcome',
             'formtitle_edit'    => 'pEdit_alias_welcome',
             'create_button'     => 'pCreate_alias_button',
-            'successmessage'    => 'pCreate_alias_result_success', # TODO: better message for edit
 
             # various settings
             'required_role' => 'admin',

Modified: trunk/model/AliasdomainHandler.php
===================================================================
--- trunk/model/AliasdomainHandler.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/model/AliasdomainHandler.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -69,9 +69,11 @@
         if ($this->new) {
             $this->msg['logname'] = 'create_alias_domain';
             $this->msg['store_error'] = 'pCreate_alias_domain_error3'; # TODO: error message could be better
+            $this->msg['successmessage'] = 'pCreate_alias_domain_success';
         } else {
             $this->msg['logname'] = 'edit_alias_domain';
             $this->msg['store_error'] = 'pCreate_alias_domain_error3'; # TODO: error message could be better
+            $this->msg['successmessage'] = 'pCreate_alias_domain_success'; # TODO: better message for edit
         }
     }
 
@@ -81,7 +83,6 @@
             'formtitle_create' => 'pCreate_alias_domain_welcome',
             'formtitle_edit' => 'pCreate_alias_domain_welcome',
             'create_button' => 'pCreate_alias_domain_button',
-            'successmessage' => 'pCreate_alias_domain_success',
 
             # various settings
             'required_role' => 'admin',

Modified: trunk/model/DomainHandler.php
===================================================================
--- trunk/model/DomainHandler.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/model/DomainHandler.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -76,9 +76,11 @@
         if ($this->new) {
             $this->msg['logname'] = 'create_domain';
             $this->msg['store_error'] = 'pAdminCreate_domain_result_error';
+            $this->msg['successmessage'] = 'pAdminCreate_domain_result_success';
         } else {
             $this->msg['logname'] = 'edit_domain';
             $this->msg['store_error'] = 'pAdminEdit_domain_result_error';
+            $this->msg['successmessage'] = 'pAdminCreate_domain_result_success'; # TODO: better message for edit
         }
     }
 
@@ -88,7 +90,6 @@
             'formtitle_create' => 'pAdminCreate_domain_welcome',
             'formtitle_edit' => 'pAdminEdit_domain_welcome',
             'create_button' => 'pAdminCreate_domain_button',
-            'successmessage' => 'pAdminCreate_domain_result_success',
 
             # various settings
             'required_role' => 'global-admin',

Modified: trunk/model/MailboxHandler.php
===================================================================
--- trunk/model/MailboxHandler.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/model/MailboxHandler.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -89,9 +89,11 @@
         if ($this->new) {
             $this->msg['logname'] = 'create_mailbox';
             $this->msg['store_error'] = 'pCreate_mailbox_result_error';
+            $this->msg['successmessage'] = 'pCreate_mailbox_result_success';
         } else {
             $this->msg['logname'] = 'edit_mailbox';
             $this->msg['store_error'] = 'pCreate_mailbox_result_error'; # TODO: better error message
+            $this->msg['successmessage'] = 'pCreate_mailbox_result_success'; # TODO: better message
         }
     }
 
@@ -107,7 +109,6 @@
             'formtitle_create' => 'pCreate_mailbox_welcome',
             'formtitle_edit' => 'pEdit_mailbox_welcome',
             'create_button' => 'pCreate_mailbox_button',
-            'successmessage' => 'pCreate_mailbox_result_success',
 
             # various settings
             'required_role' => 'admin',

Modified: trunk/model/PFAHandler.php
===================================================================
--- trunk/model/PFAHandler.php	2013-02-12 20:02:34 UTC (rev 1427)
+++ trunk/model/PFAHandler.php	2013-02-16 20:42:17 UTC (rev 1428)
@@ -8,6 +8,8 @@
     # array of error messages - if a method returns false, you'll find the error message(s) here
     public $errormsg = array();
 
+    # array of info messages (for example success messages)
+    public $infomsg = array();
 
     /**
      * variables that must be defined in all *Handler classes
@@ -341,8 +343,15 @@
         $result = $this->storemore();
 
         if ($result) {
+            # TODO: does it really make sense to skip logging if storemore() failed?
             db_log ($this->id, $this->msg['logname'], "");
+
+            # return success message
+            # TODO: use sprintf to add $this->id
+            # TODO: add option to override the success message (for example to include autogenerated passwords)
+            $this->infomsg['success'] = sprintf(Lang::read($this->msg['successmessage']), $this->id);
         }
+
         return $result;
     }
 

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.