problem with Notifable trait in user

[mohamed-alashry]

I have installed Notifynder 3.2 inside my laravel app, for try if it works I have just insert one notification :

 $user_sender_id   = 1;
 $user_receiver_id = 2;
 Notifynder::category('hello')
        ->from($user_sender_id  )
        ->to($user_receiver_id)
        ->url('http://www.yourwebsite.com/page')
        ->send();

It works properly, if I check inside database notification is there.

The problem happens when I try to get the notification in that way:

$user = User::find(2);
dd($user->getNotifications($limit = 10, $paginate = 1, $order = 'desc'));
The result is that: Notification result

The result is empty, but in database notification exist

Notification Table

and if i used
Notifynder::getAll(2,$limit = null,$paginateBool = null)
i can get them .

and i add this in my app/User.php:

use Fenos\Notifynder\Notifable;

class User extends Authenticatable
{

   use Notifable;

}

can anyone help me ?

[Gummibeer]

Can you pls try dd($user->notifications) this should output all Notifications related to this user with pure Eloquent logic.

PS: if you have started with this package I recommend you to use the newest version, ind evelopment but almost ready, dev-version-4. The documentation for this version is http://notifynder.info

In this version we have refactor everything, rewritten/reviewed every single line of code. Until now it's bugfree and ready for release, from our side, except code doc-blocks and the full documentation and some realworld tests.

[mohamed-alashry]

it returns

NotifynderCollection {#252 ▼
  #notifynderTranslator: null
  #items: []
}

p.s
my primary key for the user is users_id
is this could be the issue

[Gummibeer]

Not if you have set it as the primary key: protected $primaryKey = 'users_id';!?
Have you tested it with new v4? We have the new version in 3 internal apps in use and it's working pretty well.

[mohamed-alashry]

i tried v4 but when i try to create category this execption came up
There are no commands defined in the "notifynder:create" namespace.

[mohamed-alashry]

still stuck on this. any help..

[Gummibeer]

Yes, in the v4 is No Artisan command - Database changes belongs to migrations and seeds, there you can add your category.

[mohamed-alashry]

i found the problem, and i solve it.
if you changed the users primary key, then in vendor\fenos\notifynder\src\Notifynder\Notifable.php
change this $this->id to this $this->getKey(). and i think this is the problem .

[Gummibeer]

Good that you solved it - related to your points:

1. The polymorphic is just important if you have multiple models that use the trait
2. You can pass the mdoel itself to the from() and to() methods - we will pick the informations by getMorphClass() and getKey()
3. Oh damn ... in v4 this is solved.

These are reasons why we started with the new version cause over the time there are a lot of contributors and changes and not everything is tested with all th different options/cases/environments.
Thanks for your help to find and fix it, I will create a bugfix for it.

[mohamed-alashry]

yes i was just going to update that. but yes it was all about getkey(). glade to help