Feeling Lucky
?
?
[#25858] - Catch broken confirmConsent configuration
- Fixed in Code Base
- 22 Aug 2019
- Medium
- Build: staging
- # 25858
- Diff
- zero-24:confirmConsent
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Pull Request for Issue raised here: https://forum.joomla.de/thread/8188-dsgvo-anfrageformular-bringt-fehlermeldung/ (german)
Summary of Changes
Make sure there is an article object. When not forward to an 404 error as the article ID does not exists.
Testing Instructions
configure confirm consent
show it in the frontend
remove the article
try to show it in the frontend again
Expected result
404
Actual result
Notice: Trying to get property 'id' of non-object in /plugins/content/confirmconsent/fields/consentbox.php on line 252
Notice: Trying to get property 'language' of non-object in /plugins/content/confirmconsent/fields/consentbox.php on line 255
Notice: Trying to get property 'id' of non-object in /plugins/content/confirmconsent/fields/consentbox.php on line 269
Notice: Trying to get property 'catid' of non-object in /plugins/content/confirmconsent/fields/consentbox.php on line 269
Notice: Trying to get property 'language' of non-object in /plugins/content/confirmconsent/fields/consentbox.php on line 270
Documentation Changes Required
none
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
ChristineWk
- comment
- 20 Aug 2019
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25858.
zero-24
- comment
- 20 Aug 2019
Thanks @ChristineWk for your tests
viocassel
- comment
- 21 Aug 2019
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25858.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 21 Aug 2019
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25858.
franz-wohlkoenig
- comment
- 21 Aug 2019
Status "Ready To Commit".
zero-24
- comment
- 21 Aug 2019
Thanks
| Labels |
Added:
?
|
||
HLeithner
- comment
- 22 Aug 2019
Thank you very much for this failsafe.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-08-22 17:05:16 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
?
|
||
Confirm "Actual Result". After Patch: error 404 of previous removed article.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25858.