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:
le="color: #000000"><?php $GLOBALS['xoopsTpl']->assign('modPathIcon16', $modPathIcon16);
to:
le="color: #000000"><?php $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:
le="color: #000000"><?php 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:
le="color: #000000"><?php 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!
