User tests: Successful: Unsuccessful:
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_fields |
Labels |
Added:
?
|
I tested this and the error is now gone in the cron email. Excellent.
@jjnxpct please mark your test as successfully at Issue Tracker.
@franz-wohlkoenig : Not sure how to do that. Sorry, I'm getting lost in this GitHub stuff...
No Problem @jjnxpct. At https://docs.joomla.org/Testing_Joomla!_patches scroll to Subhead "Recording test results", this might be enough information. If not, please comment :-)
I have tested this item
Tested this on our live site where the problem occurred. After the code change the errors were gone in the cron output mail.
@franz-wohlkoenig : Thanks! Just added the successful test.
@franz-wohlkoenig : Thanks! Just added the successful test.
Hi! After updating Joomla the errors came back, because we tested this by changing a core file and now this file has been overriden by the Jomla update.
So I guess this means this has not been added to the Joomla release. Not sure how this works, but when can this 'fix' be expected in a next Joomla release? So we don't have to keep patching this ;-)
We need 2 successful tests then it can be merged in to core.
OK, thanks for letting me know. Hope someone else can test this to.
I have tested this item
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-05-28 10:35:20 |
Closed_By | ⇒ | HLeithner |
thx
@jjnxpct can you test this pr?