? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
20 Dec 2020

It is not a word count. string.truncate is a character count (and a trim to prevent partial words)

Create a newsfeed module with the following example feed https://www.nytimes.com/svc/collections/v1/publish/https://www.nytimes.com/section/world/rss.xml

Leave the character count at 0 and you have the full feed
retest with the character count at 10 and you have approx 10 characters depenant on word breaks

Pull Request for Issue #29741

avatar brianteeman brianteeman - open - 20 Dec 2020
avatar brianteeman brianteeman - change - 20 Dec 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Dec 2020
Category Language & Strings Modules Front End
avatar toivo
toivo - comment - 21 Dec 2020

I have tested this item successfully on 5f9b207

Tested successfully in Beta6-dev of 21 December.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31737.

avatar toivo toivo - test_item - 21 Dec 2020 - Tested successfully
avatar gostn gostn - test_item - 21 Dec 2020 - Tested successfully
avatar gostn
gostn - comment - 21 Dec 2020

I have tested this item successfully on 5f9b207


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31737.

avatar alikon alikon - change - 21 Dec 2020
Status Pending Ready to Commit
avatar alikon
alikon - comment - 21 Dec 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31737.

avatar infograf768
infograf768 - comment - 24 Dec 2020

This works, but I guess you are aware that this is not B/C because the field name is modified and therefore existing modules will default to 0.

Not my decision to merge.
@wilsonge @HLeithner

avatar brianteeman
brianteeman - comment - 24 Dec 2020

Good point about the changed field name. I didnt consider that. At least its only a cosmetic b/c change

I am happy to change it back but would prefer to keep the change so that the code is more meaningful

I leave it to others to decide

avatar brianteeman
brianteeman - comment - 2 Feb 2021

Please advise how to proceed to resolve and fix this bug

avatar brianteeman
brianteeman - comment - 3 Feb 2021

No point in keeping this open

avatar brianteeman brianteeman - close - 3 Feb 2021
avatar brianteeman brianteeman - change - 3 Feb 2021
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2021-02-03 19:34:17
Closed_By brianteeman
Labels Added: ? ?

Add a Comment

Login with GitHub to post a comment