Making WordPress.org

Opened 6 years ago

Closed 6 years ago

#3987 closed defect (bug) (fixed)

Only require Slack's include files if they exists

Reported by: rmarks's profile RMarks Owned by: rmarks's profile RMarks
Milestone: Priority: normal
Component: WordCamp Site & Plugins Keywords: has-patch needs-testing
Cc:

Description

When on a local vvv environment, the following error is found:

Warning: require_once(/srv/www/wordpress-meta-environment/meta-repository/wordcamp.org/public_html/wp-content/mu-plugins/includes/slack/send.php): failed to open stream: No such file or directory in /srv/www/wordpress-meta-environment/meta-repository/wordcamp.org/public_html/wp-content/mu-plugins/0-slack.php on line 9

Plan is to only require if the file exists.

Attachments (2)

3987.diff (510 bytes) - added by RMarks 6 years ago.
3987.2.diff (510 bytes) - added by RMarks 6 years ago.
Using is_readable() vs file_exists()

Download all attachments as: .zip

Change History (5)

#1 @iandunn
6 years ago

  • Owner set to RMarks

@RMarks
6 years ago

#2 @RMarks
6 years ago

  • Keywords has-patch needs-testing added

@RMarks
6 years ago

Using is_readable() vs file_exists()

#3 @coreymckrill
6 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 7960:

WordCamp: Only require Slack's include file if it exists

This avoids an error in local environments where the Slack include file
may not exist.

Fixes #3987
Props RMarks

Note: See TracTickets for help on using tickets.