[TikiWiki-commits] [Git][tikiwiki/tiki][master] [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 <69b1e38bca0a5_3b189fd483899@gitlab-sidekiq-low-urgency-cpu-bound-v2-6d4b95ff96-8lhk5.mail>

luci pushed to branch master at Tiki Wiki CMS Groupware / Tiki


Commits:
53f9afc7 by Landry Bitege at 2026-03-11T21:42:28+00:00
[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

- - - - -


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
=====================================
@@ -141,18 +141,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/53f9afc786f6b761a9ca153be776828c81c8b7a0

-- 
View it on GitLab: https://gitlab.com/tikiwiki/tiki/-/commit/53f9afc786f6b761a9ca153be776828c81c8b7a0
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.