tonyair/silverstripe-restfulserver
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-restfulserver synced 2024-10-22 14:05:58 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #49 from creative-commoners/pulls/2.0/fix-relation-getter

Browse Source 
FIX getFieldsForObj does not return relation classes in hasField() check
This commit is contained in:
Robbie Averill 2018-02-08 17:11:43 +13:00 committed by GitHub
commit b59d956143
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 11 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/DataFormatter.php
View File

@ -301,7 +301,9 @@ abstract class DataFormatter
if (is_array($this->customFields)) { if (is_array($this->customFields)) {
foreach ($this->customFields as $fieldName) { foreach ($this->customFields as $fieldName) {
// @todo Possible security risk by making methods accessible - implement field-level security // @todo Possible security risk by making methods accessible - implement field-level security
if ($obj->hasField($fieldName) || $obj->hasMethod("get{$fieldName}")) { if (($obj->hasField($fieldName) && !is_object($obj->getField($fieldName)))
|| $obj->hasMethod("get{$fieldName}")
) {
$dbFields[$fieldName] = $fieldName; $dbFields[$fieldName] = $fieldName;
} }
} }

5
tests/unit/Stubs/RestfulServerTestAuthor.php
View File

@ -2,11 +2,8 @@
namespace SilverStripe\RestfulServer\Tests\Stubs; namespace SilverStripe\RestfulServer\Tests\Stubs;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestPage;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestAuthor;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestAuthorRating;
use SilverStripe\ORM\DataObject;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
class RestfulServerTestAuthor extends DataObject implements TestOnly class RestfulServerTestAuthor extends DataObject implements TestOnly
{ {

3
tests/unit/Stubs/RestfulServerTestAuthorRating.php
View File

@ -2,9 +2,8 @@
namespace SilverStripe\RestfulServer\Tests\Stubs; namespace SilverStripe\RestfulServer\Tests\Stubs;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestAuthor;
use SilverStripe\ORM\DataObject;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
class RestfulServerTestAuthorRating extends DataObject implements TestOnly class RestfulServerTestAuthorRating extends DataObject implements TestOnly
{ {

8
tests/unit/Stubs/RestfulServerTestComment.php
View File

@ -2,12 +2,10 @@
namespace SilverStripe\RestfulServer\Tests\Stubs; namespace SilverStripe\RestfulServer\Tests\Stubs;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestPage;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestAuthor;
use SilverStripe\Security\Permission;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\PermissionProvider;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider;
/** /**
* Everybody can view comments, logged in members in the "users" group can create comments, * Everybody can view comments, logged in members in the "users" group can create comments,

4
tests/unit/Stubs/RestfulServerTestPage.php
View File

@ -2,10 +2,8 @@
namespace SilverStripe\RestfulServer\Tests\Stubs; namespace SilverStripe\RestfulServer\Tests\Stubs;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestAuthor;
use SilverStripe\RestfulServer\Tests\Stubs\RestfulServerTestComment;
use SilverStripe\ORM\DataObject;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
class RestfulServerTestPage extends DataObject implements TestOnly class RestfulServerTestPage extends DataObject implements TestOnly
{ {

4
tests/unit/Stubs/RestfulServerTestSecretThing.php
View File

@ -2,9 +2,9 @@
namespace SilverStripe\RestfulServer\Tests\Stubs; namespace SilverStripe\RestfulServer\Tests\Stubs;
use SilverStripe\Security\Permission;
use SilverStripe\ORM\DataObject;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider; use SilverStripe\Security\PermissionProvider;
class RestfulServerTestSecretThing extends DataObject implements TestOnly, PermissionProvider class RestfulServerTestSecretThing extends DataObject implements TestOnly, PermissionProvider