1
0
mirror of https://github.com/flarum/core.git synced 2025-07-15 22:06:24 +02:00
Commit Graph

10 Commits

Author SHA1 Message Date
5dc9451c21 Fix notification query with DB prefix
This was fixed in https://github.com/laravel/framework/pull/28400.
See commit 7f1048352d.
2020-05-09 14:45:57 +02:00
d492579638 Apply fixes from StyleCI
[ci skip] [skip ci]
2019-11-28 00:16:50 +00:00
7f1048352d Clean up database query
- Use existing `selectRaw()` method to avoid using the global `app()`
  helper as a service locator, which hides dependencies.
- Do the same for the join.
- The `Expression` is necessary to prevent the aliased column from being
  prefixed with the database table prefix, if configured.
2019-07-11 22:35:19 +02:00
6d14d0c39b Perform visibility checks on notification subjects at the query level
This will prevent a notification from being seen by a user if its
subject is deleted or undergoes some kind of permission change (eg.
a discussion is moved into a private tag)

ref #1380
2018-11-11 16:58:08 +10:30
aa4c4b07bd Revert notifications_from table
I didn't think this change through and it's going to be too difficult
to implement right now. It can wait until we do the notifications
revamp. For now reverting back to the old structure, with the
`sender_id` column renamed to `from_user_id`.
2018-07-21 18:35:50 +09:30
ff7f7681c7 Use Eloquent's latest and oldest 2018-07-21 17:21:08 +09:30
93b9513df2 Fix column names 2018-07-21 15:24:51 +09:30
3e3e1cbde5 fixed more attributes to match beta 8 2018-05-14 13:49:52 +02:00
a2927b725f went over most of the changed attributes from the other pr 2018-04-17 14:22:38 +02:00
b38ade986d Extract Flarum\Notification namespace 2017-10-03 18:49:53 +02:00