7
Hi Ray,
Quote:
1) Regading title length, could you please let me know
the fix so that I can apply it now. I've got a production
site and I'd like to fix this issue now.
I've gone ahead and done the following modification:
In blocks/news_top.php, I commented out the following line:
$title = $myts->makeTboxData4Show($myrow["title"]);
if (strlen($title) >= $options[2]) {
// $title = $myts->makeTboxData4Show(xoops_substr($title,0,($options[2]-1)))."...";
Here is the code you should use, in fact the line you have commented don't have to be commented, there's just a small difference inside the test itself :
Quote:
if (strlen($title) > $options[2]) {
$title = xoops_substr($title,0,$options[2]+3);
}
If think that your problem is coming from the XOOPS function named
xoops_substr, what's your version of XOOPS ? In the latest, 2.0.7, this function has been corrected. If you don't have XOOPS 2.0.7, try to upgrage this function, it can be found in the file
/xoop/include/functions.phpQuote:
That works for me for now, but I'd like to know the correct
fix. What is the array "options"? Where is options[2]
specified?
options[2] is the length of the title, it's coming from the block, that's what you have typed.
Quote:
2) Regarding the Restric View by Topic option, I did
select it, but it's not affecting my Top Stories
news items. Please advise.
Humm... yes, I saw. In fact, this block has been rewrited in the current developpment version. For this moment the title must be visible but you must have an error message when you click on the link. I'm sory but I don't want to modify the 1.2 as I'm now working on the 1.3 version. The correction will involve many other modifications. But I'm going to modify this now because the problem also exist in the current dev version. You will have to wait for the new release.... sorry (I can't give you a release date).
Bye,
Hervé