Bug tracker

This ticket has been moved to our new tracker. Open Ticket PHPBB3-8731 now.

Empty forum image causes error (fix completed in vcs)

In 3.0.5, new code was added to check for the existance of the forum_image in acp_forums.php. This check is also done if the forum_image is left blank, causing a not-found error with some installations of PHP.

Comments / History

Assigned ticket to user "bantu"

Action performed by bantu (3.0 Release Manager) on Sep 24th 2009, 12:14

Changed ticket status from "New" to "Reviewed"

Action performed by bantu (3.0 Release Manager) on Sep 24th 2009, 12:14

Posted by bantu (3.0 Release Manager) on Sep 24th 2009, 12:18

Code: Select all
Index: includes/acp/acp_forums.php
===================================================================
--- includes/acp/acp_forums.php   (revision 10183)
+++ includes/acp/acp_forums.php   (working copy)
@@ -907,7 +907,7 @@
          array('lang' => 'FORUM_TOPICS_PAGE', 'value' => $forum_data['forum_topics_per_page'], 'column_type' => 'TINT:0'),
       );
 
-      if (!file_exists($phpbb_root_path . $forum_data['forum_image']))
+      if (!empty($forum_data['forum_image']) && !file_exists($phpbb_root_path . $forum_data['forum_image']))
       {
          $errors[] = $user->lang['FORUM_IMAGE_NO_EXIST'];
       }

Posted by bantu (3.0 Release Manager) on Sep 24th 2009, 12:31

Related to Bug #PHPBB3-39005.

Linked ticket with changeset: r10184

Action performed by bantu (3.0 Release Manager) on Sep 24th 2009, 12:37

Changed ticket status from "Reviewed" to "Fix completed in SVN"

Action performed by bantu (3.0 Release Manager) on Sep 24th 2009, 12:37

Ticket details

Related SVN changesets