Feeling Lucky
?
[#27751] - [4.0] Subfield - Param repeat should be readonly
- Closed
- 1 Nov 2020
- Medium
- Build: staging
- # 27751
For what do we need this code:
// Don't allow the 'repeat' attribute to be edited
foreach ($xmlxpath->evaluate('//field[@name="repeat"]') as $field)
{
$field->setAttribute('readonly', '1');
}
In any case, it does not work.
Steps to reproduce the issue
- Create a new custom field of the type
subfield - See, that you can edit the field
repeat
Expected result
After reading the code I expected:
Field for the param repeat should be readonly.
Actual result
You can edit the field repeat.
joomla-cms-bot
- | Labels |
Added:
?
|
||
Fedik
- comment
- 1 Feb 2020
@astridx can you please check next:
1 Create a new custom field of the type subfield
2 See, that you can edit the field repeat
3 Save the field
4 Open to edit again, repeat field should be "readonly"
I found it need for prevent changing type of existing field, to avoid data loss.
If it not work, then it may be related to #27760
Original pr #24711
Fedik
- comment
- 1 Feb 2020
Thanks for checking.
Then it a valid bug.
| Status | New | ⇒ | Confirmed |
AndySDH
- comment
- 1 Nov 2020
This can be closed. It has been fixed in another PR and it's now fixed
alikon
- comment
- 1 Nov 2020
can you please reference the pr that solved this.... then i'll be very happy to close this one
| Status | Confirmed | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-11-01 12:18:39 |
| Closed_By | ⇒ | alikon |
alikon
- comment
- 1 Nov 2020
thank you
I think it was made to make "require" option to work (your issue #27736), if it not work then it cause #27736Need to find original PR and author, to be sure.