Feeling Lucky
?
[#14481] - Fix JFormFieldCaptcha
- Fixed in Code Base
- 11 Mar 2017
- Medium
- Build: 4.0-dev
- # 14481
- Diff
- joomdonation:patch-4
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
See #14467 for issue description. Look at JFormFieldCaptcha, the intention is implements JFormDomfieldinterface . However, that interface is not defined anywhere in our codebase (both Joomla 3 and Joomla 4). Since @laoneo said that the interface was abandon long time ago and this is the only Form Field has this implements JFormDomfieldinterface, I think we just need to remove the implements JFormDomfieldinterface from class definition
Testing Instructions
See #14467 and confirm the issue. Apply patch and confirm the issue fixed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
zero-24
- comment
- 11 Mar 2017
Merging on review. As it is clearly a error. Thanks @joomdonation
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-03-11 10:27:35 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
|
||
hmm looks like there was a merge conflict between staging and 4.0-dev ?
https://github.com/joomla/joomla-cms/blob/staging/libraries/cms/form/field/captcha.php#L17