Sarath
Performance and Scalability
If you've got a need for speed, this group is for you.
Beginning Developers
This space is for newcomers, who wish to build a new plugin or to customize an existing one to their liking
Elgg Technical Support
Get support from other Elgg users
Sarath replied on the discussion topic Relationship table getting bloated with notifications que- Sarath replied on the discussion topic Relationship table getting bloated with notifications que
- Sarath replied on the discussion topic Relationship table getting bloated with notifications que
- Sarath added a new discussion topic Relationship table getting bloated with notifications que in the group Plugin Development
Thanks for considering the feature request, and the clue Jerome :)
@Jerome: I looked into the code where relationship is created between the two entities and I noticed that the event is triggered after a relationship is created. Currently it is like
- A relationship is created
- If there are some event listeners which doesn't want this to happen, then the relationship is deleted from the database.
This logic seems to be wrong. Why do we need 3 unnecessary db queries - read (check existing relationship) + insert (insert new relationship) + delete (delete current relationship created as it matches our condition)? Cant we prevent the creation of relationship first place itself?
This logic seems to be wrong. Why do we need 3 unnecessary db queries - read (check existing relationship) + insert (insert new relationship) + delete (delete current relationship created as it matches our condition)? Cant we prevent the creation of relationship first place itself?
Agreed, but legacy code can sometimes be a pain ;)
- Sarath replied on the discussion topic How to add custom class names to rows when using elgg table listing?
- Sarath replied on the discussion topic How to add custom class names to rows when using elgg table listing?
- Sarath added a new discussion topic How to add custom class names to rows when using elgg table listing? in the group Plugin Development
Seems, you can only do it using custom CSS styles
I managed to achieve this by passing
'list_type' => 'my-custom-list-type',Then I copied elgg/views/default/page/components/table.php to mod/myplugin/views/default/page/components/my-custom-list-type and added my conditions in that file.
$row_attrs = ['class' => elgg_extract_class($vars, 'elgg-item', 'item_class $class_name_based_on_metadata'),];In this case, your initial code should work as is.
Try use:
'item_class' => "elgg-item $class_name_based_on_metadata",
- Sarath commented on the blog Elgg 4.0.0-beta.1 has been released