Bug #613

Error is invalid smilie path is used

Added by Joshua Mayer about 2 years ago. Updated about 2 years ago.

Status:Closed Start date:12/23/2009
Priority:Normal Due date:
Assignee:Tom Moore % Done:

100%
Category:Admin Control Panel
Target version:1.4.12
Reproducibility:Always Database Type:
Reported In MyBB Version:1.4.10 Database Version:
PHP Version:5.2.11 SQA assignments:
Browser:

Description

When adding smilies with "Add Multiple Smilies", if you enter a path that is incorrect, a PHP error is displayed.

Associated revisions

Revision 4653
Added by Tom Moore about 2 years ago

Fixes Error is invalid smilie path is used (fixes:613)

History

#1
Updated by Chris W. B. about 2 years ago

  • Status changed from New to Confirmed

Confirmed. Below is the error that is returned:

Warning [2] readdir(): supplied argument is not a valid Directory resource - Line: 303 - File: admin/modules/config/smilies.php PHP 5.2.11 (Linux)
File Line Function
[PHP] errorHandler->error
/admin/modules/config/smilies.php 303 readdir
/admin/index.php 380 require
Warning [2] closedir(): supplied argument is not a valid Directory resource - Line: 317 - File: admin/modules/config/smilies.php PHP 5.2.11 (Linux)
File Line Function
[PHP] errorHandler->error
/admin/modules/config/smilies.php 317 closedir
/admin/index.php

#2
Updated by Joshua Mayer about 2 years ago

Thought I added a screenshot... must have timed out when uploading it.

#3
Updated by Ryan Gordon about 2 years ago

  • Category set to Admin Control Panel
  • Assignee set to Tom Moore
  • Target version set to 1.4.12

#4
Updated by Tom Moore about 2 years ago

  • Status changed from Confirmed to Assigned

#5
Updated by Tom Moore about 2 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

Applied in changeset r4653.

Can a few members of SQA please test this before closing it? I couldn't replicate the PHP error on Server 2008 with PHP 5.3.0, but I could on other versions. The fix should be good though...

#6
Updated by Chris W. B. about 2 years ago

That has resolved the error for me, however maybe 2 or 3 other SQA members can double check.

#7
Updated by Michael Schlechtinger about 2 years ago

  • Status changed from Resolved to Closed

Working fine over here, too.

Also available in: Atom PDF