]> git.mxchange.org Git - mailer.git/commitdiff
Menu system fixes, list_user simplified:
authorRoland Häder <roland@mxchange.org>
Thu, 14 Jul 2011 15:38:07 +0000 (15:38 +0000)
committerRoland Häder <roland@mxchange.org>
Thu, 14 Jul 2011 15:38:07 +0000 (15:38 +0000)
- Menu entry 'add_points' has truncated output due to to short VARCHAR
- what=list_user and correspondig functions simplified towards XML-rewrite
- TODOs.txt updated

DOCS/TODOs.txt
inc/extensions/ext-menu.php
inc/libs/user_functions.php
inc/modules/admin/admin-inc.php
inc/modules/admin/what-list_user.php
inc/mysql-connect.php
templates/de/html/admin/admin_list_user.tpl
templates/de/html/admin/admin_list_user_alpha.tpl
templates/de/html/admin/admin_list_user_pagenav.tpl
templates/de/html/admin/admin_list_user_sort.tpl
templates/de/html/admin/admin_list_user_sort_form.tpl

index 89adf4e7d662fdb8068cac674eaa9d6eb53fbb93..309510000551e7dc00d00eb1d0b9ac13b77ec499 100644 (file)
@@ -54,9 +54,9 @@
 ./inc/functions.php:2214:      // @TODO Rename column data_type to e.g. mail_status
 ./inc/gen_sql_patches.php:95:// @TODO Rewrite this to a filter
 ./inc/install-functions.php:57:        // @TODO DEACTIVATED: changeDataInLocalConfigurationFile('OUTPUT-MODE', "setConfigEntry('OUTPUT_MODE', '", "');", postRequestParameter('omode'), 0);
-./inc/language/de.php:1062:    // @TODO Rewrite these two constants
-./inc/language/de.php:1078:    // @TODO Rewrite these three constants
-./inc/language/de.php:754:// @TODO Are these constants longer used?
+./inc/language/de.php:1065:    // @TODO Rewrite these two constants
+./inc/language/de.php:1081:    // @TODO Rewrite these three constants
+./inc/language/de.php:757:// @TODO Are these constants longer used?
 ./inc/language-functions.php:255:      // @TODO These are all valid languages, again hard-coded
 ./inc/language-functions.php:44:// @TODO Rewrite all language constants to this function.
 ./inc/language/newsletter_de.php:13: * @TODO This language file is completely out-dated, please do no       *
 ./inc/libs/task_functions.php:247:             // @TODO These can be rewritten to filter
 ./inc/libs/task_functions.php:51:// @TODO Move all extension-dependent queries into filters
 ./inc/libs/theme_functions.php:93:     // @TODO Can't this be rewritten to an API function?
-./inc/libs/user_functions.php:159:     // @TODO These two constants are no longer used, maybe we reactivate this code?
-./inc/libs/user_functions.php:250:// @TODO Double-check configuration entry here
-./inc/libs/user_functions.php:344:                     // @TODO Make this filter working: $ADDON = runFilterChain('post_login_update', $content);
-./inc/libs/user_functions.php:373:                     // @TODO Make this filter working: $url = runFilterChain('do_login', array('content' => $content, 'addon' => $ADDON));
+./inc/libs/user_functions.php:221:// @TODO Double-check configuration entry here
+./inc/libs/user_functions.php:315:                     // @TODO Make this filter working: $ADDON = runFilterChain('post_login_update', $content);
+./inc/libs/user_functions.php:344:                     // @TODO Make this filter working: $url = runFilterChain('do_login', array('content' => $content, 'addon' => $ADDON));
 ./inc/libs/yoomedia_functions.php:114: $response = YOOMEDIA_QUERY_API('out_textmail.php', true); // @TODO Ask Yoo!Media for test script
 ./inc/load_config.php:75:      // @TODO Rewrite them to avoid this else block
 ./inc/loader/load-extension.php:13: * @TODO Rewrite this whole file                                        *
 ./inc/modules/admin/what-list_refs.php:107:                                                            // @TODO Try to rewrite some to EL
 ./inc/modules/admin/what-list_unconfirmed.php:107:     // @TODO "Please do not call me directly." Should be rewritten to a nice selection depending on ext-bonus
 ./inc/modules/admin/what-list_unconfirmed.php:84:      // @TODO This constant might be unused? define('__LIST_UNCON_TITLE', '{--ADMIN_LIST_UNCONFIRMED_BONUS_LINKS--}');
-./inc/modules/admin/what-list_user.php:254:            // @TODO Rewrite this into a filter
-./inc/modules/admin/what-list_user.php:297:                    // @TODO Rewrite this into a filter
+./inc/modules/admin/what-list_user.php:247:            // @TODO Rewrite this into a filter
+./inc/modules/admin/what-list_user.php:290:                    // @TODO Rewrite this into a filter
 ./inc/modules/admin/what-logs.php:64:                  // @TODO Fix content-type here
 ./inc/modules/admin/what-logs.php:70:                  // @TODO Fix content-type here
 ./inc/modules/admin/what-mem_add.php:124:                              // @TODO This can be somehow rewritten to a function
 ./templates/de/html/admin/admin_list_chk_reg.tpl:1:<!-- @DEPRECATED //-->
 ./templates/de/html/admin/admin_list_country_del_row.tpl:1:<!-- @DEPRECATED //-->
 ./templates/de/html/admin/admin_list_country_edit_row.tpl:1:<!-- @DEPRECATED //-->
+./templates/de/html/admin/admin_list_user_sort_form.tpl:1:<!-- @DEPRECATED //-->
 ./templates/de/html/admin/admin_reset_pass_done.tpl:1:<!-- @DEPRECATED //-->
 ./templates/de/html/admin/admin_send_reset_link.tpl:1:<!-- @DEPRECATED //-->
 ./templates/de/html/member/member_refback_edit.tpl:1:<!-- @DEPRECATED //-->
index bb365bdcbb9e3616f4b5a444574bda212e2ffebb..c7e9681b923cc7ea183eb0d7fa73986dc48ff0a9 100644 (file)
@@ -41,10 +41,10 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Version number
-setThisExtensionVersion('0.0.3');
+setThisExtensionVersion('0.0.4');
 
 // Version history array (add more with , '0.0.1' and so on)
-setExtensionVersionHistory(array('0.0.0','0.0.1','0.0.2','0.0.3'));
+setExtensionVersionHistory(array('0.0.0','0.0.1','0.0.2','0.0.3', '0.0.4'));
 
 switch (getExtensionMode()) {
        case 'register': // Do stuff when installation is running
@@ -95,10 +95,18 @@ switch (getExtensionMode()) {
 
                        case '0.0.3': // SQL queries for v0.0.3
                                // Change menu-string
-                               addExtensionSql("UPDATE `{?_MYSQL_PREFIX?}_admin_menu` SET `title` = 'Admin-Men&uuml; editieren' WHERE `what` = 'adminedit' LIMIT 1");
+                               addExtensionSql("UPDATE `{?_MYSQL_PREFIX?}_admin_menu` SET `title`='Admin-Men&uuml; editieren' WHERE `what` = 'adminedit' LIMIT 1");
 
                                setExtensionUpdateNotes("Men&uuml;punkt &#39;Vom Admin editieren&#39; in &#39;Admin-Men&uuml; editieren&#39; bei bereits bestehenden Installationen um&auml;ndern.");
                                break;
+
+                       case '0.0.4': // SQL queries for v0.0.4
+                               // Change menu-string
+                               addExtensionSql("ALTER TABLE `{?_MYSQL_PREFIX?}_admin_menu` CHANGE `title` `title` VARCHAR(255) NOT NULL DEFAULT ''");
+                               addExtensionSql("UPDATE `{?_MYSQL_PREFIX?}_admin_menu` SET `title`='{OPEN_TEMPLATE}DEFAULT_POINTS{CLOSE_TEMPLATE} gutschreiben' WHERE `what` = 'add_points' LIMIT 1");
+
+                               setExtensionUpdateNotes("VARCHAR(50) ist zu kurz f&uuml;r Linktitel (wegen der dynamischen Anteile).");
+                               break;
                } // END - switch
                break;
 
index b7d06247d812cdb9e6b8ac03fc856538934c2753..64a95f39ef6fa149187ddd69909dc547a28ff762 100644 (file)
@@ -41,12 +41,9 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Add links for selecting some users
-function alpha ($sortby, $colspan, $return=false) {
-       if (!isGetRequestParameterSet('offset')) {
-               setGetRequestParameter('offset', 0);
-       } // END - if
-       $add = '&amp;page=' . getRequestParameter('page').'&amp;offset=' . getRequestParameter('offset');
-       foreach (array('mode','status') as $param) {
+function alpha ($sortby) {
+       $add = '';
+       foreach (array('page','offset','mode','status') as $param) {
                if (isGetRequestParameterSet($param)) {
                        $add .= '&amp;' . $param . '=' . getRequestParameter($param);
                } // END - if
@@ -62,13 +59,7 @@ function alpha ($sortby, $colspan, $return=false) {
                        $OUT .= '<strong>' . $ltr . '</strong>';
                } else {
                        // Output link to letter
-                       $OUT .= '<a href="{%url=modules.php?module=admin&amp;what=' . getWhat();
-                       foreach (array('mode','status') as $param) {
-                               if (isGetRequestParameterSet($param)) {
-                                       $OUT .= '&amp;' . $param . '=' . getRequestParameter($param);
-                               } // END - if
-                       } // END - foreach
-                       $OUT .= '&amp;letter=' . $ltr . '&amp;sortby=' . $sortby . $add . '%}">' . $ltr . '</a>';
+                       $OUT .= '<a href="{%url=modules.php?module=admin&amp;what=' . getWhat() . '&amp;letter=' . $ltr . '&amp;sortby=' . $sortby . $add . '%}">' . $ltr . '</a>';
                }
 
                if ((($counter / getConfig('user_alpha')) == round($counter / getConfig('user_alpha'))) && ($counter > 0)) {
@@ -80,23 +71,18 @@ function alpha ($sortby, $colspan, $return=false) {
 
        // Prepare content
        $content = array (
-               'colspan2'        => $colspan,
                'alpha_selection' => $OUT
        );
 
        // Load template
        $OUT = loadTemplate('admin_list_user_alpha', true, $content);
-       if ($return === true) {
-               // Return generated code
-               return $OUT;
-       } else {
-               // Output generated code
-               outputHtml($OUT);
-       }
+
+       // Return generated code
+       return $OUT;
 }
 
 // Add links for sorting
-function addSortLinks ($letter, $sortby, $colspan, $return=false) {
+function addSortLinks ($letter, $sortby) {
        $OUT = '';
        if (!isGetRequestParameterSet('offset')) setGetRequestParameter('offset', 0);
        if (!isGetRequestParameterSet('page'))   setGetRequestParameter('page'  , 0);
@@ -137,82 +123,67 @@ function addSortLinks ($letter, $sortby, $colspan, $return=false) {
                }
        } // END - foreach
 
-       // Add list and colspan
+       // Add output
        $content['list'] = substr($OUT, 0, -1);
-       $content['colspan2'] = $colspan;
 
        // Load template
        $OUT = loadTemplate('admin_list_user_sort', true, $content);
 
-       // Should we return or output?
-       if ($return === true) {
-               // Return code
-               return $OUT;
-       } else {
-               // Output code
-               outputHtml($OUT);
-       }
+       // Return code
+       return $OUT;
 }
 
 // Add page navigation
-function addPageNavigation ($numPages, $offset, $showForm, $colspan, $return=false) {
-       // @TODO These two constants are no longer used, maybe we reactivate this code?
-       //if ($showForm === true) {
-       //      // Load form for changing number of lines
-       //      define('__FORM_HEADER', loadTemplate('admin_list_user_sort_form', true));
-       //      define('__FORM_FOOTER', '<tr><td colspan="'.$colspan.'" class="seperator bottom">&nbsp;</td></tr>');
-       //} else {
-       //      // Empty row
-       //      define('__FORM_HEADER', '<tr><td colspan="' . $colspan . '" class="seperator">&nbsp;</td></tr>');
-       //      define('__FORM_FOOTER', '<tr><td colspan="' . $colspan . '" class="seperator bottom">&nbsp;</td></tr>');
-       //}
-
+function addPageNavigation ($numPages) {
+       // Start with empty content
        $OUT = '';
-       for ($page = 1; $page <= $numPages; $page++) {
-               if (($page == getRequestParameter('page')) || ((!isGetRequestParameterSet('page')) && ($page == 1))) {
-                       $OUT .= '<strong>-';
-               } else {
-                       if (!isGetRequestParameterSet('letter')) setGetRequestParameter('letter', '');
-                       if (!isGetRequestParameterSet('sortby')) setGetRequestParameter('sortby', 'userid');
 
-                       // Base link
-                       $OUT .= '<a href="{%url=modules.php?module=admin&amp;what=' . getWhat();
+       // Create only the navigation if page count > 1
+       if ($numPages > 1) {
+               // Create navigation links for every page
+               for ($page = 1; $page <= $numPages; $page++) {
+                       if (($page == getRequestParameter('page')) || ((!isGetRequestParameterSet('page')) && ($page == 1))) {
+                               $OUT .= '<strong>-';
+                       } else {
+                               if (!isGetRequestParameterSet('letter')) setGetRequestParameter('letter', '');
+                               if (!isGetRequestParameterSet('sortby')) setGetRequestParameter('sortby', 'userid');
 
-                       // Add status/mode
-                       foreach (array('mode','status') as $param) {
-                               if (isGetRequestParameterSet($param)) {
-                                       $OUT .= '&amp;' . $param . '=' . getRequestParameter($param);
-                               } // END - if
-                       } // END - foreach
+                               // Base link
+                               $OUT .= '<a href="{%url=modules.php?module=admin&amp;what=' . getWhat();
 
-                       // Letter and so on
-                       $OUT .= '&amp;letter=' . getRequestParameter('letter') . '&amp;sortby=' . getRequestParameter('sortby') . '&amp;page=' . $page . '&amp;offset=' . $offset . '%}">';
-               }
+                               // Add status/mode
+                               foreach (array('mode','status') as $param) {
+                                       if (isGetRequestParameterSet($param)) {
+                                               $OUT .= '&amp;' . $param . '=' . getRequestParameter($param);
+                                       } // END - if
+                               } // END - foreach
+
+                               // Letter and so on
+                               $OUT .= '&amp;letter=' . getRequestParameter('letter') . '&amp;sortby=' . getRequestParameter('sortby') . '&amp;page=' . $page . '&amp;offset=' . getConfig('user_limit') . '%}">';
+                       }
 
-               $OUT .= $page;
+                       $OUT .= $page;
 
-               if (($page == getRequestParameter('page')) || ((!isGetRequestParameterSet('page')) && ($page == 1))) {
-                       $OUT .= '-</strong>';
-               } else  {
-                       $OUT .= '</a>';
-               }
+                       if (($page == getRequestParameter('page')) || ((!isGetRequestParameterSet('page')) && ($page == 1))) {
+                               $OUT .= '-</strong>';
+                       } else  {
+                               $OUT .= '</a>';
+                       }
 
-               if ($page < $numPages) $OUT .= '|';
-       } // END - for
+                       if ($page < $numPages) {
+                               $OUT .= '|';
+                       } // END - if
+               } // END - for
 
-       // Remember the list and colspan
-       $content['list']     = $OUT;
-       $content['colspan2'] = $colspan;
+               // Add list output
+               $content['list'] = $OUT;
 
-       // Load template
-       $OUT = loadTemplate('admin_list_user_pagenav', true, $content);
-       if ($return === true) {
-               // Return code
-               return $OUT;
-       } else {
-               // Output code
-               outputHtml($OUT);
-       }
+               // Load template
+               $OUT = loadTemplate('admin_list_user_pagenav', true, $content);
+       } // END - if
+
+       // Return code
+       return $OUT;
 }
 
 // Create email link to user's account
index 18f99003fc417a312f80e05394738a35b803b343..b7537c8b604212d74f8b9bc9236c88aa6f78b9d0 100644 (file)
@@ -626,9 +626,9 @@ function generateUserProfileLink ($userid, $title = '', $what = 'list_user') {
        if (($title == '') && (isValidUserId($userid))) {
                // Set userid as title
                $title = $userid;
-       } elseif ($userid == 0) {
+       } elseif (!isValidUserId($userid)) {
                // User id zero is invalid
-               return '<strong>' . $userid . '</strong>';
+               return '<strong>' . makeNullToZero($userid) . '</strong>';
        }
 
        if (($title == '0') && ($what == 'list_refs')) {
index 0c861764522ffd981c4a34001ed85d600346c553..58bc853aa343bef54e59715280e4cb8f08023691 100644 (file)
@@ -45,7 +45,6 @@ addYouAreHereLink('admin', __FILE__);
 
 // Init variables
 $MORE = '';
-$colspan = 4;
 
 // Add lock reason?
 if (isExtensionInstalledAndNewer('user', '0.3.5')) {
@@ -238,17 +237,11 @@ LIMIT 1",
        // Are there some entries?
        if (!SQL_HASZERONUMS($result_master)) {
                // We have some (new?) registrations!
-               $templateContent['colspan2']   = ($colspan + 2);
                $templateContent['user_count'] = getTotalConfirmedUser();
 
-               // Sorting links
-               $templateContent['alpha_sort'] = alpha(getRequestParameter('sortby'), ($colspan + 1), true);
-               $templateContent['sort_links'] = addSortLinks(getRequestParameter('letter'), getRequestParameter('sortby'), ($colspan + 1), true);
-               $templateContent['page_nav']   = '';
-
-               if ($numPages > 1) {
-                       $templateContent['page_nav'] = addPageNavigation($numPages, getConfig('user_limit'), true, $colspan, true);
-               } // END - if
+               // Sorting links and number of pages
+               $templateContent['sort_links'] = addSortLinks(getRequestParameter('letter'), getRequestParameter('sortby'));
+               $templateContent['num_pages'] = $numPages;
 
                // Column with nickname when nickname extension is present
                // @TODO Rewrite this into a filter
index 818bdc0c06f335862b7d24660783bc69b7d47d60..8ab0fd974f965a01490c21d904cbeaf3f9aa3b60 100644 (file)
@@ -100,7 +100,7 @@ if ((!isInstalling()) && (!isInstallationPhase())) {
 
                        // Is it a valid resource?
                        if (SQL_SELECT_DB($GLOBALS['mysql']['dbase'], __FILE__, __LINE__) === true) {
-                               // This is required for extension 'optimize' to work
+                               // Set database name (required for ext-optimize and isSqlTableCreated())
                                setConfigEntry('__DB_NAME', $GLOBALS['mysql']['dbase']);
 
                                // Remove MySQL array from namespace
index c341708166e8e191028a397ddd8f7363dabd70ea..883d894667102660e0c69fab25cb21acf38f8c17 100644 (file)
@@ -1,14 +1,14 @@
 <div align="center">
 <table border="0" cellspacing="0" cellpadding="0" class="table dashed">
        <tr>
-               <td class="table_header bottom" colspan="$content[colspan2]" align="center">
+               <td class="table_header bottom" colspan="5" align="center">
                        <strong>{%template,AdminListUserTitle%}:</strong>
                </td>
        </tr>
-       $content[alpha_sort]
+       {%pipe,getRequestParameter,alpha=sortby%}
        $content[sort_links]
-       $content[page_nav]
+       {%pipe,addPageNavigation=$content[num_pages]%}
        $content[rows]
-       $content[page_nav]
+       {%pipe,addPageNavigation=$content[num_pages]%}
 </table>
 </div>
index 89082addcafb3646f9627af3b61e6c0f051739d1..a88ee35cfd951456a2babaaa9c2b812e353f4e95 100644 (file)
@@ -1,5 +1,5 @@
 <tr>
-       <td colspan="$content[colspan2]" align="center">
+       <td colspan="5" align="center">
        <table border="0" cellspacing="0" cellpadding="0" width="510">
                <tr>
                        <td class="admin_alpha">
index d823cc8cb1fe2ac88c423005883b7438d480852d..9fa1c2b889bf648275a0b7e48f5725c747ae748a 100644 (file)
@@ -1,5 +1,5 @@
 <tr>
-       <td class="table_header" colspan="$content[colspan2]" align="center">
+       <td class="table_header" colspan="5" align="center">
                <table border="0" cellspacing="0" cellpadding="0">
                <tr>
                        <td class="admin_pagenav">
index b67e1b5a2023a332315231a607a40340922dc86c..98364b3a62655f8a4d17c3aa93f34622df64e2e6 100644 (file)
@@ -1,5 +1,5 @@
 <tr>
-       <td class="bottom" colspan="$content[colspan2]" align="center">
+       <td class="bottom" colspan="5" align="center">
        <table border="0" cellspacing="0" cellpadding="0">
                <tr>
                        <td class="admin_sortlinks">
index 45b10fd81e55922590d7a3e8922089e86073dcad..11f5d85d80b1ca4d16cf4c19e14af24e5ec2a60f 100644 (file)
@@ -1,14 +1 @@
-<tr>
-       <td colspan="$content[colspan2]" align="center">
-       <table border="0" cellspacing="0" cellpadding="0">
-               <tr>
-                       <td align="center">
-                       <!--&nbsp;
-                               <form accept-charset="UTF-8" action="{%url=modules.php?module=admin&amp;what=list_user%}" method="post">
-                               </form>
-                       //-->
-                       </td>
-               </tr>
-       </table>
-       </td>
-</tr>
+<!-- @DEPRECATED //-->