? Pending

User tests: Successful: Unsuccessful:

avatar saharin88
saharin88
20 Oct 2019

Ignore records created by blocked users.

avatar saharin88 saharin88 - open - 20 Oct 2019
avatar saharin88 saharin88 - change - 20 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2019
Category Libraries
avatar mbabker
mbabker - comment - 21 Oct 2019

Arbitrarily not loading categories from the tree because a user account is blocked seems wrong.

avatar saharin88
saharin88 - comment - 21 Oct 2019

Arbitrarily not loading categories from the tree because a user account is blocked seems wrong.

Мore precisely, do not count records (in category) from blocked users*
This change is probably more related to the public part of the site. For example, a category module.

avatar alikon
alikon - comment - 21 Oct 2019

or at least quite debatable, imho, can you describe a real use case scenario for this ?

avatar saharin88
saharin88 - comment - 21 Oct 2019

or at least quite debatable, imho, can you describe a real use case scenario for this ?

Any Counter of materials of a category, whether it is a module of a category or something else - a custom code (extends Joomla\CMS\Categories\Categories)

avatar mbabker
mbabker - comment - 21 Oct 2019

Then implement the logic in your subclass instead of arbitrarily changing the entirety of core. This really comes across as a total edge case that should be dealt with in that specific implementation.

Imagine if menu items or articles or modules started disappearing because you blocked the user account that "owns" the item.

avatar HLeithner
HLeithner - comment - 22 Oct 2019

Thanks @saharin88 for your PR but I see this as michael, it's an edge case. You can solve this by setting a special viewlevel if a user is blocked per batch processing for articles.

I'm closing this.

avatar HLeithner HLeithner - change - 22 Oct 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-10-22 13:27:20
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 22 Oct 2019
avatar saharin88
saharin88 - comment - 22 Oct 2019

Thanks @saharin88 for your PR but I see this as michael, it's an edge case. You can solve this by setting a special viewlevel if a user is blocked per batch processing for articles.

I'm closing this.

Ок

Add a Comment

Login with GitHub to post a comment