User tests: Successful: Unsuccessful:
This PR solves a bug with the word count parameter of mod_feed. If you load a feed with HTML tags (for example: http://community.joomla.org/blogs/community.feed?type=rss) and set the word count to 20 there are way less than 20 words visible. The two things that cause this problem:
JHtml::_('string.truncate')
counts the characters, not the words of the string<a href="http://link.nl">
are counted tooThis PR change the text of the parameter to Character count
and I also adds strip_tags()
for the character count.
Other changes in this PR are code style and some change to make the code more readably and logic.
Status | New | ⇒ | Pending |
Labels |
Added:
?
?
|
Category | ⇒ | Front End RTL |
Labels | |||
Easy | No | ⇒ | Yes |
This PR has received new commits.
CC: @Ruchiranga
This PR has received new commits.
CC: @Ruchiranga
@Ruchiranga Thanks for testing! Yes you're right, the RTL function wasn't working, I used a wrong variable. I also changed the label of the parameter to Character limit
, that is a better label indeed.
Can you please test again, and confirm the problems are solved?
I have tested this item successfully on 607a440
* The amount of characters shown has become better. See
Before applying the patch and
After applying the patch.
I´m not sure if this breaks something on existing sites because even if the wordcount didn´t work right before
i don´t know if we can just change it to character limit?
Isn´t it better to fix the wordcount instead?
Changing from word count to character count will break peoples sites. Imagine I have it set to 10
my news feed of approximately ten words excepting html bug
my news fe
So with this change form word to character count it cannot be accepted.
Status | Pending | ⇒ | Information Required |
Labels |
If you want to update the PR to retain the word count then this can be tested otherwise I am afraid it will have to be closed
Labels |
Category | Front End RTL | ⇒ | Front End Language & Strings RTL |
Status | Information Required | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-05-08 09:39:03 |
Closed_By | ⇒ | brianteeman |
Closed for the reason above
Labels |
I have tested this item unsuccessfully on 61f6253
### Tested successfully
Tested unsuccessfully
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8230.