91
zyspec
Re: Comments per page / News
  • 2017/5/25 22:44

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@Hadesteam,

This looks like it may be a core bug - if it's what I think it is...

Can you try something to see if this fixes the issue you're seeing:

in ./include/comment_view.php around line 76 change:
$comments $comment_handler->getByItemId($xoopsModule->getVar('mid'), $com_itemid$com_dborder);

to:
$comments $comment_handler->getByItemId($xoopsModule->getVar('mid'), $com_itemid$com_dbordernull, (int)$comment_config['comments_pager']);


Then look at your page with the comments mode set to "Flat" to see if this fixes the problem.



92
zyspec
Re: Problems with Soapbox
  • 2017/5/20 15:25

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Quote:
This is work in progress, i.e. things will change, so use it on a production site.

I think @mamba meant to say DON'T use this on a production site...



93
zyspec
Re: XoopsTube 1.07 Beta and Xoops 2.5.9 beta
  • 2017/5/13 22:57

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Until Bleekk can get a chance to update the files you can change line 48 in the ./sql/mysql.sql file from:
description  TEXT            NOT NULL DEFAULT '',

to:
description  TEXT         NOT NULL,


which should let you install the module...



94
zyspec
Re: Xoopspoll
  • 2017/5/10 14:45

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Sorry, I missed that you had included that one above...

just wanted to make sure you found them all.



95
zyspec
Re: Xoopspoll
  • 2017/5/9 22:25

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


You'll find the same issue in ./blocks/singlepoll.php around line 120
if (XoopspollConstants::MULTIPLE_SELECT_POLL === $pollVars['multiple']) {

should be:
if (XoopspollConstants::MULTIPLE_SELECT_POLL === (int) $pollVars['multiple']) {


These regressions were introduced in commit 8bf75c0 by introduction of exact (type) checking.



96
zyspec
Re: Publisher Random Item Image
  • 2017/5/6 12:43

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Sorry @Zap_English

Thanks @Trabis....



97
zyspec
Re: Publisher Random Item Image
  • 2017/5/5 16:54

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@Zap_English,

Unfortunately it's a little more complex than just adding anything that's been previously posted in this thread. The random item block does not currently display the main item image, just the item itself.

Let me state: I have not tried any of the following code, I don't have the module currently loaded on my test site and I'm sorry but I don't have time right now to go try/test this. So anything you see/use below is "at your own risk"... Before you try this code I'd recommend you go to your XOOPS Admin > Preferences > System Options > General Settings and set Debug Mode to Enable debug (inline mode).

Basically what needs to happen is that in ./blocks/items_random_item.php you will need to get the item's main image thumbnail, if it doesn't exist then substitute you're default image. Then in the ./templates/blocks/publisher_items_random_item.tpl you will need to display the image.

The publisher_items_random_item_show() function in ./blocks/items_random_item.php should look something like:
/**
 * @param $options
 *
 * @return array
 */
function publisher_items_random_item_show($options)
{
    
$block     = array();
    
$publisher PublisherPublisher::getInstance();
    
// creating the ITEM object
    
$itemsObj $publisher->getHandler('item')->getRandomItem('', array(PublisherConstants::PUBLISHER_STATUS_PUBLISHED));
    if (!
is_object($itemsObj)) {
        return 
$block;
    }
    
$mainImage $itemsObj->getMainImage();
    if (empty(
$mainImage['image_thumb'])) {
        
// the  in the following line path_to_your_default_image_thumbnail should be something 
        // like XOOPS_UPLOADS_URL . "publisher/default_thumbnail.jpg"
        
$mainImage['image_thumb'] = {path_to_your_default_image_thumbnail};
    } 
    
$block['content']       = $itemsObj->getBlockSummary(300true); //show complete summary but truncate to 300 if only body available
    
$block['id']            = $itemsObj->itemid();
    
$block['url']           = $itemsObj->getItemUrl();
    
$block['title']         = $itemsObj->title(); // irmtfan
    
$block['lang_fullitem'] = _MB_PUBLISHER_FULLITEM;
    
$block['item_thumb']    = $mainImage['item_thumb'];
    return 
$block;
}


Now, provided you're using the standard templates (i.e. they haven't been modified) you should be able to use the following in your ./templates/blocks/publisher_items_random_item.tpl file:
<{$block.content}><br><br>
<
br class="clear">
<
div class="floatleft">
    <
img class="pad3" src="<{$block.item_thumb}>" alt="" title="<{$block['title']}>">
</
div>
<
div align="right">
    <
a href='<{$block.url}>'><{$block.lang_fullitem}></a>
</
div>



98
zyspec
Re: xsitemap alternative theme
  • 2017/5/5 15:45

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


I just issued a pull request to fix the sub-category problem with the XML rendering. You can see the changes athttps://github.com/XoopsModules25x/xsitemap/pull/7/commits/f54007c2d7730d89301436b6101e11ead630dbc3

The changes that effect the sub-cateogory issue are contained in the ./include/function.php file. The changes in the xoops_version.php file aren't required, I just updated the date so we could tell that the BETA 1 had been updated.



99
zyspec
Re: xsitemap alternative theme
  • 2017/5/5 14:49

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


EDIT: Can I ask if you've tried turning off subcategories in the module's Preferences to see how the search engines react?

I think I see the problem with sub-categories... the XML isn't being formed correctly when there are subcategories.



100
zyspec
Re: Publisher Random Item Image
  • 2017/5/4 17:03

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Two (2) things...

1. The original code you posted isn't anywhere in the copy of the file you just sent, so I'm still confused.
2. beverages_items_random_item_show isn't a standard part of the publisher module so I'm not sure where you got it. Perhaps you should go back to the person that modified the code and see if they can help you figure it out.




TopTop
« 1 ... 7 8 9 (10) 11 12 13 ... 100 »



Login

Who's Online

181 user(s) are online (110 user(s) are browsing Support Forums)


Members: 0


Guests: 181


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