? ? Failure
Pull Request for # 7611

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
8 May 2016

Pull Request for Issue #7611

Summary of Changes

Moves the application's default logs directory to administrator/logs, changes include:

  • Updated .gitignore
  • Updated the build script to only include the default directory in the full (new) install package; updates and patches won't get this
  • The sysinfo screen uses the new path as the default lookup for the log path
  • The install app uses the new path as the default setting

B/C implications

I ain't got a clue to be honest. Since Joomla isn't installable as an extension of itself anymore, the issues I raised previously regarding that on the linked issue shouldn't be a problem. I guess the worst thing is if someone's got a hardcoded JPATH_ROOT . '/logs' log path in their scripts.

Testing Instructions

On a new install of Joomla, the default log path should be administrator/logs.

When updating to a Joomla version including this package, the administrator/logs directory should NOT be added to your filesystem.

The log_path configuration parameter is NOT changed at update, a site owner can/should apply this change at their discretion once accepted.

avatar mbabker mbabker - open - 8 May 2016
avatar mbabker mbabker - change - 8 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 8 May 2016
Rel_Number 0 7611
Relation Type Pull Request for
avatar brianteeman brianteeman - change - 8 May 2016
Category Feature Request Installation
avatar brianteeman brianteeman - change - 8 May 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 11 May 2016
Milestone Added:
avatar wilsonge wilsonge - close - 11 May 2016
avatar wilsonge wilsonge - merge - 11 May 2016
avatar wilsonge wilsonge - reference | c2de752 - 11 May 16
avatar wilsonge wilsonge - merge - 11 May 2016
avatar wilsonge wilsonge - close - 11 May 2016
avatar wilsonge wilsonge - change - 11 May 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-11 21:20:21
Closed_By wilsonge
avatar wilsonge
wilsonge - comment - 11 May 2016

Merged on review

avatar mbabker mbabker - head_ref_deleted - 11 May 2016

Add a Comment

Login with GitHub to post a comment