71
zyspec
Re: Wfdownloads 3.30 Beta-1 for XOOPS 2.5.9 and PHP 5.6+
  • 2018/6/11 0:34

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Unfortunately it looks like Mamba forgot to rewrite part of the install function in the ./include/oninstall.php file. You might try replacing lines 41-64 (the xoops_module_pre_install_wfdownloads function) with:
function xoops_module_pre_install_wfdownloads(XoopsModule $module)
{
    include 
__DIR__ '/common.php';
    
/** @var XoopsModulesWfdownloadsUtility $utility */
    
$utility = new XoopsModulesWfdownloadsUtility();
    
//check for minimum XOOPS version
    
$xoopsSuccess $utility::checkVerXoops($module);
    
// check for minimum PHP version
    
$phpSuccess   $utility::checkVerPhp($module);
    if (
false !== $xoopsSuccess && false !==  $phpSuccess) {
        
$moduleTables =& $module->getInfo('tables');
        foreach (
$moduleTables as $table) {
            
$GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' $GLOBALS['xoopsDB']->prefix($table) . ';');
        }
    }
    return 
$xoopsSuccess && $phpSuccess;
}


That should at least get you past the pre-install check for PHP and XOOPs versions.



72
zyspec
Re: countdown module for XOOPS 2.5.9 ?
  • 2018/6/8 15:21

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@ultraman147,

You're correct. I apologize, I missed 2 more places that need to be changed.

In the same file (./templates/countdown_index.tpl) around line 26 you will see:
<a href="../index.php?op=edit&amp;id=<{$event.id}>"><{$event.name}></a>

please change it to:
<a href="./index.php?op=edit&amp;id=<{$event.id}>"><{$event.name}></a>


and similarly around line 46 you will see:
<a href="../index.php?op=add"><{$smarty.const._CD_TEXT_ADD_EVENT}></a>

please change to:
<a href="./index.php?op=add"><{$smarty.const._CD_TEXT_ADD_EVENT}></a>


I hope this helps.



73
zyspec
Re: countdown module for XOOPS 2.5.9 ?
  • 2018/6/5 14:18

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


I haven't tried this but it looks to me like you should be able to change ./templates/countdown_index.tpl from:
<tr>
+        <
td class="head">
+            <
a href="../index.php?sort=name&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_NAME}></a>
+        </
td>
+        <
td class="head">
+            <
a href="../index.php?sort=description&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_DESCRIPTION}></a>
+        </
td>
+        <
td class="head">
+            <
a href="../index.php?sort=enddatetime&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_EXPIRATION}></a>
+        </
td>
+        <
td class="head">
+            <
a href="../index.php?sort=remaining&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_REMAINING}></a>
+        </
td>
+    </
tr>

to:
<tr>
+        <
td class="head">
+            <
a href="./index.php?sort=name&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_NAME}></a>
+        </
td>
+        <
td class="head">
+            <
a href="./index.php?sort=description&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_DESCRIPTION}></a>
+        </
td>
+        <
td class="head">
+            <
a href="./index.php?sort=enddatetime&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_EXPIRATION}></a>
+        </
td>
+        <
td class="head">
+            <
a href="./index.php?sort=remaining&order=<{$cd_sort_order}>"><{$smarty.const._CD_TEXT_REMAINING}></a>
+        </
td>
+    </
tr>


All this change does is removes an 'extra' period in the href links to make sure it references the current relative directory. This should fix the 'missing' /countdown directory reference.



74
zyspec
Re: Publisher and URL rewrite method
  • 2017/12/22 0:16

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Logged this issue in GitHub so it can be tracked... seehttps://github.com/XoopsModules25x/publisher/issues/88



75
zyspec
Re: Not Executing: function xoops_module_pre_install_dirname()
  • 2017/12/5 22:03

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@wishcraft,

I haven't checked to see if the pre_install function is being called but the line you called out above is perfectly valid. The "double brackets" (curly braces) {} is only there for code clarity and does not affect the string's interpretation by the PHP compiler. If the string's within the double-quote then the brackets are preferred by most PHP coding standards.

In fact, it makes it much easier to read if you want to include an array element in the string directly. Something like:
$myVar "This is a string that identifies a value from the array:  {$someArray['anIndex']}";



76
zyspec
Re: White Screen when accessing modules as admin
  • 2017/12/5 0:42

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Are you still getting the same error in themes/xbootstrap/xoscripts/breadcrumb.php?



77
zyspec
Re: White Screen when accessing modules as admin
  • 2017/12/4 18:03

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@blackrx

You might try and replace:
// groups 
if( is_object( @$xoopsUser ) ) { 
    
$member_handler =xoops_gethandler'member' ) ; 
    
$groups $member_handler->getGroupsByUser$xoopsUser->getVar('uid') , true ) ; 
    foreach( 
$groups as $group ) { 
        
$groups4assign[] = array( 'id' => $group->getVar('groupid') , 'name' => $group->getVar('name') ) ; 
    } 
} else { 
    
$groups4assign[] = array( 'id' => XOOPS_GROUP_ANONYMOUS 'name' => _GUESTS ) ;

$this->assign"xugj_groups" $groups4assign ) ;

with:
// groups 
$groups4assign = array();
if (
$GLOBALS['xoopsUser'] instanceof XoopsUser)) {
    
$member_handler xoops_gethandler('member'); 
    
$groups $member_handler->getGroupsByUser($GLOBALS['xoopsUser']->getVar('uid') , true);
    if (
is_array($groups)) {
        foreach( 
$groups as $group ) {
            
$groups4assign[] = array('id' => $group->getVar('groupid') , 'name' => $group->getVar('name')); 
        } 
    }
}
if (empty(
$groups4assign) { 
    
$groups4assign[] = array('id' => XOOPS_GROUP_ANONYMOUS 'name' => _GUESTS);

$this->assign'xugj_groups' $groups4assign ) ;


This code change initializes a couple of variables that may not be initialized properly. Let us know if this helps.



78
zyspec
Re: Example of Using Vue.js in Blocks
  • 2017/11/21 1:55

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


I think it's a great idea - it should be a side block onhttps://xoops.org/modules/publisher/category.php?categoryid=1

We should have a similar block for modules (XoopsModules25x) onhttps://xoops.org/modules/publisher/category.php?categoryid=2

And for themes (Xoopsthemes) on
https://xoops.org/modules/publisher/category.php?categoryid=3



79
zyspec
Re: White Screen when accessing modules as admin
  • 2017/11/21 1:46

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


You might also want to check the Protector module logs to see if anything shows up there...



80
zyspec
Re: XOOPS Modules Tutorials in ReadTheDocs style
  • 2017/8/9 14:15

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@mamba - this is potentially a really good opportunity, thanks for seeking this out...

It appears that once ReadTheDocs is setup correctly (and our DocBooks are configured) that ReadTheDocs will 'easily' support both localization/internationalization, auto-updating using WebHooks, versioning support, etc.

I'm sure there's a whole set of rules for things we need to do (or not do) so if anyone wants to help document best practices I'm sure @mamba would love the help!




TopTop
« 1 ... 5 6 7 (8) 9 10 11 ... 100 »



Login

Who's Online

247 user(s) are online (171 user(s) are browsing Support Forums)


Members: 0


Guests: 247


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Apr 30
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits