? Success

User tests: Successful: Unsuccessful:

avatar Arkadiy-Sedelnikov
Arkadiy-Sedelnikov
13 Oct 2014

Trigger a need for content filtering.

avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - open - 13 Oct 2014
avatar jissues-bot jissues-bot - change - 13 Oct 2014
Labels Added: ?
avatar brianteeman brianteeman - change - 14 Oct 2014
Category Components Plugins
avatar joomline
joomline - comment - 15 Oct 2014

Good innovation. Allows you to expand the use of com_content.

avatar Bakual
Bakual - comment - 15 Oct 2014

Please have a look at #4308 which tries to unify plugin triggers.
Maybe by coordinate you can achieve more :smile:

avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - change - 15 Oct 2014
Title
Add trigger to com_content category model
Add trigger to com_content front category model
avatar Arkadiy-Sedelnikov
Arkadiy-Sedelnikov - comment - 25 Nov 2014

See "MiniCCK Filter" on this page http://demoj3.argens.ru/index.php/blog
Its work with this trigger. Good or bad?

avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - close - 25 Nov 2014
avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - close - 25 Nov 2014
avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - change - 25 Nov 2014
Title
Add trigger to com_content category model
Add trigger to com_content front category model
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-11-25 17:38:19
avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - change - 25 Nov 2014
Status Closed New
avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - reopen - 25 Nov 2014
avatar Arkadiy-Sedelnikov Arkadiy-Sedelnikov - reopen - 25 Nov 2014
avatar zero-24 zero-24 - change - 27 Nov 2014
Status New Pending
avatar roland-d roland-d - change - 20 Aug 2015
Status Pending Information Required
avatar roland-d
roland-d - comment - 20 Aug 2015

Hello @Arkadiy-Sedelnikov

Thank you for your contribution.

The last comment here was on 25th November. So the question is, Is this issue/pull request still valid? Does the PR linked by Bakual solve the issue?

If no reply is received within 4 weeks we will close this issue.

Thanks for understanding!


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

avatar Arkadiy-Sedelnikov
Arkadiy-Sedelnikov - comment - 21 Aug 2015

I already solved the problem of the absence of the trigger in my extension substitution model. All it'd be better if the trigger was in the home model, but the urgent need for this anymore.

avatar mbabker
mbabker - comment - 21 Aug 2015

I'm confused on what exactly this is supposed to be doing.

avatar FPerisa FPerisa - test_item - 21 Aug 2015 - Tested successfully
avatar FPerisa
FPerisa - comment - 21 Aug 2015

Testing the trigger was successful. The trigger calls the function "onGetContentItems" for a content plugin like "contact".

avatar Arkadiy-Sedelnikov
Arkadiy-Sedelnikov - comment - 21 Aug 2015

See http://demoj3.argens.ru/index.php/blog

Module "MiniCCK Filter" working on this trigger.

avatar mbabker
mbabker - comment - 21 Aug 2015

I get that the trigger is added and running. What I don't get is why it's being added and what the purpose is for this very specific use case. Generally, we do not have event triggers to manipulate an object (model or database query typically) before a data query is executed; this changes that behavior for one very exclusive use case. Right now this PR looks to me like a "I need this feature for my extension so I'm going to add it without anymore detail" situation and my inclination is to suggest it be closed because of that (looks to be something for an edge case in an extension and no clarifying explanation about why this is proposed). IMO, this PR needs to be better explained; why is this event trigger needed for only this one use case out of all the components shipped with the Joomla core.

avatar Arkadiy-Sedelnikov
Arkadiy-Sedelnikov - comment - 23 Aug 2015

I wrote that for my extension this PR is not mandatory, I already made a spoof of this model with a built-in trigger. The trigger I use for content filtering ин additional fields on my CCK, so it can use other CCK parasitic on com_content.

avatar Bakual
Bakual - comment - 23 Aug 2015

I'm closing this PR as in the current state it will not get merged. To many questions are open.

If we want to introduce this new plugin event, it should better be placed in a library model somewhere so it's triggered for all extensions, not a single one.

avatar Bakual Bakual - change - 23 Aug 2015
Status Information Required Closed
Closed_Date 2014-11-25 17:38:19 2015-08-23 17:35:54
Closed_By Bakual
avatar Bakual Bakual - close - 23 Aug 2015

Add a Comment

Login with GitHub to post a comment