model_permission_check instanceof Member not namespaced

colymba / silverstripe-restfulapi

SilverStripe RESTful API with a default JSON serializer.

BSD 3-Clause "New" or "Revised" License

64 stars 34 forks source link

model_permission_check instanceof Member not namespaced #97

Closed lukeskelding closed 5 years ago

lukeskelding commented 5 years ago

There is a check on line 548 of RESTfulAPI.php:

https://github.com/colymba/silverstripe-restfulapi/blob/b24b7eb0c139a83a9c5cc9194a780e8882914c44/src/RESTfulAPI.php#L548

This checks the authenticator owner is a member instance. This reference to Member hasn't been namespaced for SS4 (SilverStripe\Security\Member) so its never true meaning the model level checks fail.

colymba commented 5 years ago

should be fixed by #99