?
avatar brianteeman
brianteeman
3 Jun 2021

The query for articles in these two modules intentionally does not include archived articles.

However when calculated the emptystate the query must include archived.

To test. Change the state of all articles to archived and check these two modules. They are displaying "No Articles have been created yet." which is not correct

avatar brianteeman brianteeman - open - 3 Jun 2021
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 3 Jun 2021
avatar PhilETaylor
PhilETaylor - comment - 3 Jun 2021

LMAO And this is why my first PR was right... but you opposed it. to the point I changed it.

avatar brianteeman
brianteeman - comment - 3 Jun 2021

Where? There is nothing in that link remotely related

avatar PhilETaylor
PhilETaylor - comment - 3 Jun 2021

Where? There is nothing in that link remotely related

Im sorry you feel that way... but once again you are wrong.

if you had not opposed it the text would have remained

"You do not have any popular articles yet"
"You do not have any recent articles yet" 

The "Empty State" of these modules is not "There are no articles, you have not ever created any"... That is the empty state of the component...

These modules are not aware of the FULL table of content, they are only aware if there is something to show or not... its not a real "empty state" like the "empty state" is defined for the extensions, it more a "text string when there is no items to show"... and therefore the original language like those below is "more correct" and that is what #33472 proposed.

"You do not have any popular articles yet"
"You do not have any recent articles yet"

And the two articles strings are NOT trying to say that there are no articles in the database, they are trying to say that the purpose of the module is a empty state, and as the data for these modules is about Popular and Recently added articles, the language has to reflect that underlying dataset - there are no popular articles and there are no recent articles.

avatar PhilETaylor
PhilETaylor - comment - 3 Jun 2021

And that was the whole purpose originally of the textSuffix

'textSuffix' => '_RECENT',

'textSuffix' => '_POPULAR',

Which again, you just removed earlier today /facepalm

avatar brianteeman
brianteeman - comment - 3 Jun 2021

Sorry but the text of the string is nothing to do with the issue. An article can be popular and still be archived. An article can be recently created and still be archived.

avatar PhilETaylor
PhilETaylor - comment - 3 Jun 2021

An article can be popular and still be archived. An article can be recently created and still be archived.

Yes, but Even before empty state, these modules have NEVER shown archived items...

Even before empty state, these modules would have shown nothing...

avatar PhilETaylor
PhilETaylor - comment - 3 Jun 2021

To test. Change the state of all articles to archived and check these two modules. They are displaying "No Articles have been created yet." which is not correct

So what exactly is your EXPECTED behaviour?

Maybe

"You do not have any non-archived popular articles yet"
"You do not have any non-archived recent articles yet" 
avatar ReLater
ReLater - comment - 3 Jun 2021

Maybe a good chance to think about a take-back of these strange emptystate codes from at least modules; with additional DB queries and JLayouts for nothing than a message.

avatar PhilETaylor
PhilETaylor - comment - 4 Jun 2021

with additional DB queries

Empty State for modules DOESN'T run any additional DB Queries at all.

Empty State for Extensions ONLY runs a single additional DB query when NO ITEMS are about to be shown.

On a normal site, with data in the db, Empty State Design DOESN'T have any overhead whatsoever ever.

Until @brianteeman answers the question, "what exactly is your EXPECTED behaviour" there is nothing that can be achieved on this issue.

avatar brianteeman brianteeman - change - 24 Jun 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-06-24 09:23:42
Closed_By brianteeman
avatar brianteeman brianteeman - close - 24 Jun 2021

Add a Comment

Login with GitHub to post a comment