? Pending

User tests: Successful: Unsuccessful:

avatar C-Lodder
C-Lodder
17 Aug 2017

Pull Request for Issue #17576

Summary of Changes

Fixes the PHP error being throw on com_redirect

Testing Instructions

Enable the redirect plugin and then try to open com_redirect

avatar joomla-cms-bot joomla-cms-bot - change - 17 Aug 2017
Category Administration com_redirect
avatar C-Lodder C-Lodder - open - 17 Aug 2017
avatar C-Lodder C-Lodder - change - 17 Aug 2017
Status New Pending
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 17 Aug 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2017

I have tested this item successfully on 1977de8


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17587.

avatar brianteeman brianteeman - test_item - 17 Aug 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 17 Aug 2017

I have tested this item successfully on 1977de8

Thanks for the quick fix


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17587.

avatar zero-24 zero-24 - change - 17 Aug 2017
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 17 Aug 2017

RTC. MErge onhold until @wilsonge is ready with the staging => 4.0-dev merge. Thnaks!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17587.

avatar wilsonge
wilsonge - comment - 17 Aug 2017

This has been fixed in the 4.0 merge because that part of the code got refactored with the 3.8 move to use modal for editing the plugin.

avatar wilsonge wilsonge - change - 17 Aug 2017
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2017-08-17 21:59:28
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 17 Aug 2017
avatar wilsonge
wilsonge - comment - 17 Aug 2017

Also wrong milestone @zero-24 :P

avatar zero-24
zero-24 - comment - 17 Aug 2017

Thnaks for correcting me. @wilsonge

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 18 Aug 2017

Should RTC set again?

avatar zero-24
zero-24 - comment - 18 Aug 2017

Should RTC set again?

@franz-wohlkoenig why? @wilsonge fixed that via a direct commit already.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 18 Aug 2017

@zero-24 RTC was set and i didn't see it. So if nothing gets lost all fine.

Add a Comment

Login with GitHub to post a comment