Page MenuHomePhabricator

new 'include_path' setting should incorporate old setting
Closed, ResolvedPublic

Description

Author: bugzilla

Description:
LocalSettings.php includes this line:

ini_set( "include_path", ".;$IP;$IP/includes;$IP/languages" );

However, for email confirmations, PEAR is used (never mind there's no mention of
that in the install docs!) and its directory also needs to be included.

I put this in my file, and it seems like a reasonable thing to have in the
standard install:

$include_path = ini_get( "include_path" );
ini_set( "include_path", "$include_path;$IP;$IP/includes;$IP/languages" );

Version: 1.5.x
Severity: normal

Details

Reference
bz4134

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 8:57 PM
bzimport set Reference to bz4134.
bzimport added a subscriber: Unknown Object (MLST).

I'll just point out that PEAR::Mail use is optional and you must manually turn it on; at that point you
can also change your include_path.

Normally PHP's built-in mail() function is used, which requires no such inclusion.

  • Bug 3250 has been marked as a duplicate of this bug. ***

robchur wrote:

This was done in SVN trunk, r14087.