Menu

#138 return message after module specific install script

Patches
closed
5
2012-09-25
2005-11-10
Bruno
No

During module specific install script, i needed to have
a message feet back.
This hack return message fom the module specific
install script.

In the
modules\system\admin\modulesadmin\modulesadmin.php
programme
Line 470 in 2.0.13 :
Line 287 in 2.2.3.a
I have replaced
if (function_exists('xoops_module_install_'.$dirname)) {
$func = 'xoops_module_install_'.$dirname;
if (!$func($module)) {
$msgs[] = 'Failed to execute '.$func;
} else {
$msgs[] = ''.$func.' executed
successfully.';
}
}

by this code

if (function_exists('xoops_module_install_'.$dirname)) {
$func = 'xoops_module_install_'.$dirname;
$msg_install=$func($module);
if (!$msg_install) {
$msgs[] = 'Failed to execute '.$func;
} else {
$msgs[] = $msg_install.'
'.$func.'
executed successfully.';
}
}

Discussion

  • Bruno

    Bruno - 2005-11-10

    2.0.13\modules\system\admin\modulesadmin\modulesadmin.php

     
  • Skalpa Keo

    Skalpa Keo - 2006-01-19

    Logged In: YES
    user_id=882380

    By adding an additional test so the msg is displayed only if
    it's a string (and not a boolean), it should be ok.
    I'll try that.

    Thx

     
  • Bruno

    Bruno - 2006-01-19

    Logged In: YES
    user_id=1093822

    You the king !

    Merci

     
  • Skalpa Keo

    Skalpa Keo - 2006-05-13

    Logged In: YES
    user_id=882380

    Done on branch #126903, to be released in 2.0.14/2.2.5

     

Log in to post a comment.