PHP Notices on Joomdle Users Screen

  • Jonathan
  • Topic Author
  • Offline
  • New Member
  • New Member
More
14 years 7 months ago #1 by Jonathan
PHP Notices on Joomdle Users Screen was created by Jonathan

Notice: Undefined variable: image in ...\administrator\components\com_joomdle\views\users\tmpl\default.php on line 67
Notice: Undefined variable: text in ...\administrator\components\com_joomdle\views\users\tmpl\default.php on line 67


See lines 66,67,68.
Putting @ before $text and $image removed the messages.

Using PHP 5.3 - Messages only appear if E_INFO enabled in php.ini.

Please Log in or Create an account to join the conversation.

More
14 years 7 months ago #2 by Antonio Durán
Replied by Antonio Durán on topic Re: PHP Notices on Joomdle Users Screen
Thank you very much for the report.
Will check into it as soon as I can, as I want to release a new version, with the short tags changes applied along today, so if you see any other thing, please comment.

Antonio

Please Log in or Create an account to join the conversation.

  • Jonathan
  • Topic Author
  • Offline
  • New Member
  • New Member
More
14 years 7 months ago #3 by Jonathan
Replied by Jonathan on topic Re: PHP Notices on Joomdle Screens
Similary:

Warning: Invalid argument supplied for foreach() in .../administrator/components/com_joomdle/helpers/applications.php on line 122


Suggest changing to:

if (!empty($pc)) foreach(...) { ... }

Please Log in or Create an account to join the conversation.

  • Jonathan
  • Topic Author
  • Offline
  • New Member
  • New Member
More
14 years 7 months ago - 14 years 7 months ago #4 by Jonathan
Replied by Jonathan on topic Re: PHP Notices on Joomdle Screens - 2
Errors on Course Requests tab of admin.

Notice: Undefined variable: c in ...\administrator\components\com_joomdle\views\courseapplications\view.html.php on line 128

Add following before foreach(...) on line 121 $c = array();

Warning: Invalid argument supplied for foreach() in ...\administrator\components\com_joomdle\views\courseapplications\tmpl\default.php on line 21

Add following before foreach(...) on line 21 if (!empty($this->courses))

Using PHP 5.3 - Messages only appear if E_INFO enabled in php.ini.
Last edit: 14 years 7 months ago by Jonathan.

Please Log in or Create an account to join the conversation.