Quote:
Thanks, Timgno! This time I was able to install it without any problems.
So far, I see two problems:
1) In /admin/modules.php you need to change from:
$GLOBALS['xoopsTpl']->assign('modPathIcon16', $modPathIcon16);
to:
$GLOBALS['xoopsTpl']->assign('modPathIcon16', TDMC_URL.'/'.$modPathIcon16);
otherwise the images are not visible, since XOOPS is looking for them in /admin/icons/16
2) You still didn't fix the issue with with the case when we don't assign any name to a table prefix. Since it is "optional", I don't assign anything, and for several months now I am getting this error:
Parse error: syntax error, unexpected ')', expecting variable (T_VARIABLE) or '$' in C:/wamp64/www/257tdmc191/modules/zumba3/index.php on line 34
with the code being:
if ($categoriesCount > 0) {
$categoriesAll = $categoriesHandler->getAllCategories();
include_once XOOPS_ROOT_PATH . '/class/tree.php';
$mytree = new XoopsObjectTree($categoriesAll, 'id', 'parent');
foreach (array_keys($categoriesAll) as $)
It seems like you want to assign the prefix to the variable, but if it's not there, you're getting only the "$" sign:
Quote:
foreach (array_keys($categoriesAll) as $)
So please add to your test cases a scenario where the user doesn't choose to add a prefix for the table...
But overall it is looking better! Congratulations!
And thank you for keeping working on this!
