User tests: Successful: Unsuccessful:
Pull Request for Issue #36016.
As title says. For both tag modules.
Use more than 20 tags to yor articles,
Set the maximum to a number > 20 and make sure that the tags are displayed corrctly in frontend
see #36016
Any number is possible
Status | New | ⇒ | Pending |
Category | ⇒ | Modules Front End |
Clearly visible that the maximum count is 20
After:
Now user can set the value of the maximum count.
I have tested this item
I have tested this item
I followed the test instructions and the patch worked as described.
Status | Pending | ⇒ | Ready to Commit |
RTC
@HLeithner requested valid changes. please remove RTC
oops missed that
@HLeithner default is 5 as in other modules. The value 0 is not really useful, of course, as the result then would be empty, but for mysql this is not a problem.
If I change it here ( 0 = all), we must change it in all other modules too.
Labels |
Added:
?
|
@HLeithner default is 5 as in other modules. The value 0 is not really useful, of course, as the result then would be empty, but for mysql this is not a problem. If I change it here ( 0 = all), we must change it in all other modules too.
That's the reason I requested the implementation for this case in php (check for 0 and remove the setlimit call)
Made the changes as requested by @HLeithner . Only if the input is > 0, the maximum number is set as limit. Could you please re-test? @BertaOctech @Shubhamverma2796 ? Thanks!
Status | Ready to Commit | ⇒ | Pending |
Back to pending.
The min attribute in the number field is not validated. Also with min=0 it is possible to enter every negative value. This is not in scope of this PR.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-11-16 17:49:55 |
Closed_By | ⇒ | chmst |
@Shubhamverma2796 would you like to open an issue for the bug you found in joomla? #36019 (comment)
Maybe you also find a solution ;)
@chmst In the above video, you can see that I've fixed the issue; whenever the user tries to input any -ve value, the value automatically gets changed to 0. In this way, we can easily prevent the user from entering any negative value.
The problem is I am new here and not well-versed with the PHP and code structure of Joomla. Therefore, I cannot figure out where(in which file or folder) I should put the function written by me so that everything works fine. If anyone can guide me, then it would be my pleasure.
Thanks
@Shubhamverma2796 javascript is not the solution
The min attribute in the number field is not validated. Also with min=0 it is possible to enter every negative value. This is not in scope of this PR.
Just add validate="number" and this will ensure that the min=0 is validated
if you do it this way you have to modify the php code to allow "0" as no limit option.