No Code Attached Yet
avatar pwireless
pwireless
12 Nov 2021

Steps to reproduce the issue

Create new Module Tags-Popular. It's not allowed to show more than 20 tags.

Expected result

Show more than 20 tags. Have a high limit e.g. 500.

Actual result

System information (as much as possible)

Additional comments

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar pwireless pwireless - open - 12 Nov 2021
avatar pwireless pwireless - change - 12 Nov 2021
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 12 Nov 2021
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 12 Nov 2021
avatar brianteeman
brianteeman - comment - 12 Nov 2021

It wouldnt be "popular" tags if there were 500

avatar chmst
chmst - comment - 12 Nov 2021

In other modules (polular articles, most read articles) we use type=number and leave it to the user how many itemshe wants to see.

avatar brianteeman
brianteeman - comment - 12 Nov 2021

It has had a maximum of 20 since Elin introduced the code 9 years ago - just saying

avatar HLeithner
HLeithner - comment - 12 Nov 2021

I see no technical reason for the limit. if you write a pull request to remove this restriction I would support it.

avatar chmst chmst - change - 12 Nov 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-11-12 10:52:37
Closed_By chmst
avatar chmst chmst - close - 12 Nov 2021
avatar chmst
chmst - comment - 12 Nov 2021

Please test #36019

avatar chmst chmst - change - 16 Nov 2021
Status Closed New
Closed_Date 2021-11-12 10:52:37
Closed_By chmst
avatar chmst chmst - reopen - 16 Nov 2021
avatar richard67
richard67 - comment - 8 Dec 2021

@chmst Why reopened? Isn't there a pull request which claims to fix it?

avatar Quy
Quy - comment - 8 Dec 2021

@richard67 it was reopened before #36261.

avatar richard67 richard67 - change - 8 Dec 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-12-08 21:47:49
Closed_By richard67
avatar richard67 richard67 - close - 8 Dec 2021
avatar richard67
richard67 - comment - 8 Dec 2021

Ah … ok.

Closing as having a pull request. Please test #36261 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment