11
Cesagonchu
Re: News 1.67 Final released for XOOPS 2.5.5

Strangely, I have again the presence of bugs that you had corrected here Mamba:

Warningarray_keys() expects parameter 1 to be array, null given in file /class/xoopsform/grouppermform.php line 129

WarningInvalid argument supplied for foreach() in file /class/xoopsform/grouppermform.php line 129

WarningInvalid argument supplied for foreach() in file /class/xoopsform/grouppermform.php line 253

Warningarray_keys() expects parameter 1 to be array, null given in file /class/xoopsform/grouppermform.php line 266

12
irmtfan
Re: News 1.67 Final released for XOOPS 2.5.5
  • 2012/7/9 5:10

  • irmtfan

  • Module Developer

  • Posts: 3419

  • Since: 2003/12/7


i cant find any new bug.
I update the news module of my own website without any issue.

@Cesag:
I cant reproduce your bugs.
in the news/admin/groupperms.php i dont have any issue. is this the link that you have those errors?

13
Cesagonchu
Re: News 1.67 Final released for XOOPS 2.5.5

Quote:

irmtfan wrote:
in the news/admin/groupperms.php i dont have any issue. is this the link that you have those errors?


Yes, but Mamba corrected these bugs in the past, and I didn't have problems after that.

I wanted to try this link :
http://sourceforge.net/projects/xoops/files/XOOPS%20Module%20Repository/XOOPS2/News/

I Downloaded XOOPS2.55_mod_news_1.67_xoops.zip, and bugs came back!

I do my tests with Xoops 2.5.5, English version here, fresh install and on dedicated server :

Quote:
Apache Version Apache/2.2.22 (Ubuntu)
Minimum PHP required: 5.2 (your version is 5.3.10-1ubuntu3.2)
MYSQL minimum version required: 5.0.7 (your version is 5.5.24-0ubuntu0.12.04.1)
Minimum XOOPS required: 2.5 (your version is 2.5.5)
Minimum ModuleAdmin required: 1.1 (your version is 1.1)



14
Mamba
Re: News 1.67 Final released for XOOPS 2.5.5
  • 2012/7/10 5:00

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Quote:
Yes, but Mamba corrected these bugs in the past, and I didn't have problems after that.

I did check the download file, and the SVN, and they are pretty much the same, so there should be no issue. Can you compare you code to the one on SVN?

Maybe there is a conflict with another module? Or did you do any updates to the Core?

Try to deactivate other modules, and see if the error is still there?
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

15
Cesagonchu
Re: News 1.67 Final released for XOOPS 2.5.5

I found the problem. I have these errors as I did not create a Topic.

16
Mamba
Re: News 1.67 Final released for XOOPS 2.5.5
  • 2012/7/10 6:36

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Quote:
I found the problem. I have these errors as I did not create a Topic.

This should not happened. The same as with Post/Edit News, when there is not Topic, News should redirect you to Topic Tab, with an error message.

I'll fix it...
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

17
Cesagonchu
Re: News 1.67 Final released for XOOPS 2.5.5

Thank you

18
Mamba
Re: News 1.67 Final released for XOOPS 2.5.5
  • 2012/7/10 7:56

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


It's now fixed in XOOPS SVN

Download: XOOPS File Repository
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

19
black_beard
Re: News 1.67 Final released for XOOPS 2.5.5

A bug in the creation of PDF document (makepdf.php) when using UTF-8 charset. Appearance of odd characters in the PDF.

To fix this bug

Replace line 120 of file : makepdf.php
$pdf->WriteHTML($pdf_config['slogan'], $pdf_config['scale']);


by

if ( _CHARSET == 'UTF-8') { 
    
$pdf->WriteHTML(utf8_decode($pdf_config['slogan']), $pdf_config['scale']); 
} else {
    
$pdf->WriteHTML($pdf_config['slogan'], $pdf_config['scale']);
}


and replace line 140 of file : makepdf.php
$out=NEWS_PDF_AUTHOR.': ';


by

if ( _CHARSET == 'UTF-8') { 
    
$out=utf8_decode(NEWS_PDF_AUTHOR).': '
} else {
    
$out=NEWS_PDF_AUTHOR.': '
}

20
Mamba
Re: News 1.67 Final released for XOOPS 2.5.5
  • 2012/7/11 18:00

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Thanks! Fixed in XOOPS SVN

In the near future, we'll need to convert News from fpdf to tcpdf, so all modules are consistently using tcpdf, and we can provide tcpdf as part of XOOPS library.
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

Login

Who's Online

214 user(s) are online (142 user(s) are browsing Support Forums)


Members: 0


Guests: 214


more...

Donat-O-Meter

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

Latest GitHub Commits