? Success

User tests: Successful: Unsuccessful:

avatar bertmert
bertmert
18 Apr 2015

Test:

  • Add a custom path for cache directory in your configuration.php:
public $cache_path = '/absolute/path/to/my/joomla34/mycache';
  • Go to Backend > System > System Information > Directory Permissions
  • You'll see cache and administrator/cache marked as "Cache Directory". Not path entered above that doesn't exist yet and should be marked Unwritable.

cache-before

  • Apply patch:

  • Afterwards you'll see correct path of cache directory like entered in configuration.php.

cache-after

  • If you want to test also column Status (Writable/Unwritable) create custom cache directory now.
avatar bertmert bertmert - open - 18 Apr 2015
avatar joomdonation
joomdonation - comment - 19 Apr 2015

@test: Success.

avatar zero-24 zero-24 - change - 29 Apr 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 29 Apr 2015
Labels Removed: ?
avatar zero-24 zero-24 - change - 29 Apr 2015
Labels Added: ?
avatar zero-24 zero-24 - alter_testresult - 29 Apr 2015 - joomdonation: Tested successfully
avatar zero-24 zero-24 - change - 29 Apr 2015
Category Administration Components
avatar zero-24 zero-24 - change - 29 Apr 2015
Status New Pending
avatar Fedik Fedik - test_item - 16 May 2015 - Tested successfully
avatar Fedik
Fedik - comment - 16 May 2015

test works good,
but still need to fix the merge conflict

avatar izharaazmi
izharaazmi - comment - 16 May 2015

@bertmert When already talking about this, we should consider that the same concept should be applicable for tmp and log paths as well.

avatar bertmert
bertmert - comment - 16 May 2015

@Fedik

Merge conflict? I don't understand. Something for me to do here?

avatar bertmert
bertmert - comment - 16 May 2015

@izharaazmi
Yes, but
Because there's an open discussion #6579 (without decision yet) I thought about to combine both issues but in the end "I was too scared" ;-)
So, I think, should be another PR.

avatar zero-24
zero-24 - comment - 11 Jun 2015

@bertmert

Merge conflict? I don't understand. Something for me to do here?

Yes. Do you use git via commandline or GUI?

Maybe a rebase can fix it. Else the easiest way to fix it is to open a new PR against staging with a clean branch. As we have 2 successfull test we can set the new or the fixed PR RTC :smile:

avatar zero-24
zero-24 - comment - 11 Jun 2015

Closing as we have a new PR thanks @bertmert

avatar zero-24 zero-24 - change - 11 Jun 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-06-11 21:45:34
Closed_By zero-24
avatar zero-24 zero-24 - close - 11 Jun 2015

Add a Comment

Login with GitHub to post a comment