| # |
Jun 24th 2021, 08:44 |
 kevin.pfeifer |
e.g. the global Model.afterSave event is being triggered here https://github.com/cakephp/cakephp/blob/master/src/ORM/Table.php#L2009 |
| # |
Jun 24th 2021, 08:43 |
slackebot |
coupling is the measure of how little a class is “wired” to external objects, and how much that class is depending on them." |
| # |
Jun 24th 2021, 08:43 |
conehead |
Like the first sentence of the Event System says: "Creating maintainable applications is both a science and an art. It is well-known that a key for having good quality code is making your objects loosely coupled and strongly cohesive at the same time. Cohesion means that all methods and properties for a class are strongly related to the class itself and it is not trying to do the job other objects should be doing, while loosely |
| # |
Jun 24th 2021, 08:38 |
slackebot |
test it |
| # |
Jun 24th 2021, 08:38 |
conehead |
@erwane What is weird about it? I think it is even cleaner to seperate code like this. Especially if multiple models are involved @kevin.pfeifer Yes thanks, that is what I think I will do (or what I am already doing). Was just wondering if these events were triggered automatically. I just don't see code of updating other data or sending mails in my table. Especially by putting it in an EventListener it makes it way easier to |
| # |
Jun 24th 2021, 08:36 |
kevin.pfeifer |
but I can't tell you the performance impact of one over the other |
| # |
Jun 24th 2021, 08:35 |
kevin.pfeifer |
but sure, you could dispatch an event in the afterSave function of your UsersTable and then put your logic in the listener for `Model.Users.afterSave` |
| # |
Jun 24th 2021, 08:30 |
kevin.pfeifer |
look at the last code example in this section https://book.cakephp.org/4/en/core-libraries/events.html#registering-anonymous-listeners |
| # |
Jun 24th 2021, 08:30 |
erwane |
But it's ... weird ... |
| # |
Jun 24th 2021, 08:29 |
erwane |
If you want events in another file, you have to trigger you own events name. You have to do ```public function afterSave($event) { $this->dispatchEvent('Model.Users.afterSave', ...) } ``` |
| # |
Jun 24th 2021, 08:29 |
conehead |
Or I would catch 'Model.afterSave' and then check if its the correct Table...but probably that would slow things down a lot if I check it after every save |
| # |
Jun 24th 2021, 08:27 |
conehead |
Yes, but in that case I would have to dispatch a new event in the afterSave method, correct? |
| # |
Jun 24th 2021, 08:26 |
kevin.pfeifer |
usually you are fine adding these events to your config/bootstrap.php (or require a events.php in the config/bootstrap.php which adds those listeners) |
| # |
Jun 24th 2021, 08:25 |
kevin.pfeifer |
well ok, the afterSave() function in the UsersTable is just a preconfigured function called on that event you want. But if you want that to be present elsewhere you just need to be sure, that your listener is set before the actuall event is being triggered |
| # |
Jun 24th 2021, 08:23 |
conehead |
@kevin.pfeifer not sure...should it? In my opinion it is cleaner to seperate it. I do not want to blow up the UsersTable |
| # |
Jun 24th 2021, 08:22 |
conehead |
Was hoping I could do something like this in the AppController: ``` EventManager::instance()->on( 'Model.Users.afterSave', function ($event) { debug($event); die(); } );``` |
| # |
Jun 24th 2021, 08:21 |
kevin.pfeifer |
well shouldn't that code be present in the `afterSave()` function present in the UsersTable? |
| # |
Jun 24th 2021, 08:20 |
kevin.pfeifer |
@romuald see https://stackoverflow.com/questions/58327906/cakephp-pagination-how-do-i-sort-by-the-count-of-a-contained-model |
| # |
Jun 24th 2021, 08:20 |
conehead |
On global level I want to react to a specific Table event. Lets say I want to add an Listener to 'Model.Users.afterSave' to send new Users an E-Mail |
| # |
Jun 24th 2021, 08:19 |
erwane |
you can listen Global events but Model.afterSave will be global |
| # |
Jun 24th 2021, 08:19 |
kevin.pfeifer |
well you definitely could rename and manually connect tables and entities yourself but in my opinion this just increases the work needed to be done and future errors to pop up I would always stick to https://book.cakephp.org/4/en/intro/conventions.html and prevent as many reserved words (wherever they are present - SQL, PHP or wherever) as possible |
| # |
Jun 24th 2021, 08:18 |
erwane |
Events are attached to an instance. So, if you are doing actions on UsersTable, the Model.afterSave will call usersTable::afterSave |
| # |
Jun 24th 2021, 08:17 |
conehead |
Good morning everyone. Anyone using events a lot? I was just wondering...can I just listen for events for a specific table? Aren't they called automatically? I know they trigger for example 'Model.afterSave', but they do not trigger spefic events like 'Model.Users.afterSave'? Do I have to trigger them manually? |
| # |
Jun 24th 2021, 08:16 |
erwane |
and shool_groupes ? |
| # |
Jun 24th 2021, 08:16 |
erwane |
sessions can be workshops ? |
| # |
Jun 24th 2021, 08:15 |
alamnaryab |
I can you any name, but wanted to learn how experts do it. I like the prefixed version `school_classes` `academic_sessions` |
| # |
Jun 24th 2021, 08:12 |
erwane |
@alamnaryab you can't use another approching names ? |
| # |
Jun 24th 2021, 08:12 |
kevin.pfeifer |
@alamnaryab I would rename the table to something like `custom_classes` or `project_classes` or something that is related to what this table actually does. Then you have a valid name and everything should get auto-generated with the bake command |
| # |
Jun 24th 2021, 07:53 |
romuald |
Uploaded file: https://uploads.kiwiirc.com/files/7c36d290f9028c62f3e120f811158060/pasted.txt |
| # |
Jun 24th 2021, 07:52 |
alamnaryab |
what is best practice in this scenario? should I name tables as `clases` instead of `classes` but then label will be wrong |
| # |
Jun 24th 2021, 07:51 |
romuald |
``` |
| # |
Jun 24th 2021, 07:51 |
romuald |
<th scope="col"><?= $this->Paginator->sort('Alerts.alerts_total', "Total alerts") ?></th> |
| # |
Jun 24th 2021, 07:51 |
romuald |
/ Template/Products/index.ctp |
| # |
Jun 24th 2021, 07:51 |
romuald |
```php |
| # |
Jun 24th 2021, 07:51 |
romuald |
``` |
| # |
Jun 24th 2021, 07:51 |
romuald |
$this->set(compact('products')); |
| # |
Jun 24th 2021, 07:51 |
romuald |
$products = $this->paginate($products); |
| # |
Jun 24th 2021, 07:51 |
romuald |
]); |
| # |
Jun 24th 2021, 07:51 |
romuald |
} |
| # |
Jun 24th 2021, 07:51 |
romuald |
])->where(['Alerts.active' => true]); |
| # |
Jun 24th 2021, 07:51 |
romuald |
'alerts_total' => $q->func()->count('Alerts.id') |