[TikiWiki-commits] [Git][tikiwiki/tiki][29.x] [BP][ENH][FIX] Custom Modules: Display parameters form instead of plain old...

"luci \(@luciash\) via TikiWiki-cvs" <[email protected]>
Newsgroups gmane.comp.cms.tiki.cvs
Message-ID <69b476e4c6e22_3b18bdac154@gitlab-sidekiq-low-urgency-cpu-bound-v2-fdfffcb87-q95kp.mail>

luci pushed to branch 29.x at Tiki Wiki CMS Groupware / Tiki


Commits:
04d7f3c7 by Landry Bitege at 2026-03-13T20:35:28+00:00
[BP][ENH][FIX] Custom Modules: Display parameters form instead of plain old textarea for custom (user) modules too
---
* [ENH][FIX] Custom Modules: Display parameters form instead of plain old textarea for custom (user) modules too
---
* [FIX] Custom Modules: Display parameters form for custom modules when assigning

See merge request tikiwiki/tiki!9658

(cherry picked from commit 53f9afc786f6b761a9ca153be776828c81c8b7a0)

See merge request tikiwiki/tiki!9744

- - - - -


2 changed files:

- templates/admin_modules_form.tpl
- tiki-admin_modules.php


Changes:

=====================================
templates/admin_modules_form.tpl
=====================================
@@ -86,7 +86,7 @@
                     {/if}
                 </fieldset>
             {/tab}
-            {if isset($assign_info.type) and $assign_info.type eq 'function'}
+            {if isset($assign_info.params) and !empty($assign_info.params)}
                 {foreach from=$assign_info.params key=sect item=params}
                     {tab name="{$sect|capitalize}" key="{$sect}"}
                         <fieldset id="param_section_{$sect}">


=====================================
tiki-admin_modules.php
=====================================
@@ -137,18 +137,17 @@ if (! empty($_REQUEST['edit_assign'])) {
     }
 
     $modinfo = $modlib->get_module_info($info['name']);
-    if ($modinfo['type'] != 'function') {
-        $smarty->assign_by_ref('assign_rows', $info['rows']);
-        $smarty->assign_by_ref('assign_params', $info['params']); // For old-style (user) modules
-    } else {
-        if (empty($info['params'])) {
-            $info['params'] = [];
-        }
-        $modlib->dispatchValues($info['params'], $modinfo['params']);
-        if (isset($modinfo['params']['rows'])) {
-            $modinfo['params']['rows']['value'] = $info['rows'];
+    $params = [];
+
+    if (! empty($info['params'])) {
+        if (is_array($info['params'])) {
+            $params = $info['params'];
+        } else {
+            parse_str($info['params'], $params);
         }
     }
+
+    $modlib->dispatchValues($params, $modinfo['params']);
     $smarty->assign('assign_info', $modinfo);
 }
 //post is used for preview because there is another submit item on the form requiring it



View it on GitLab: https://gitlab.com/tikiwiki/tiki/-/commit/04d7f3c7a1f73c4e93f8ff75cc31e317db33b7f4

-- 
View it on GitLab: https://gitlab.com/tikiwiki/tiki/-/commit/04d7f3c7a1f73c4e93f8ff75cc31e317db33b7f4
You're receiving this email because of your account on gitlab.com. Manage all notifications: https://gitlab.com/-/profile/notifications | Help: https://gitlab.com/help

_______________________________________________
TikiWiki-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tikiwiki-cvs
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.