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

Merge pull request #9447 from mattclegg/docs__GridFieldDetailForm_ItemRequest-httpError

Browse Source 
[DOCS] Better debug text for errors generated by GridFieldDetailForm_ItemRequest
This commit is contained in:
Robbie Averill 2020-04-02 13:05:49 -07:00 committed by GitHub
commit f77f725355
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 24 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
composer.json
View File

@ -55,7 +55,8 @@
"require-dev": { "require-dev": {
"phpunit/phpunit": "^5.7", "phpunit/phpunit": "^5.7",
"sminnee/phpunit-mock-objects": "^3.4.5", "sminnee/phpunit-mock-objects": "^3.4.5",
"silverstripe/versioned": "^1" "silverstripe/versioned": "^1",
"squizlabs/php_codesniffer": "^3.5"
}, },
"provide": { "provide": {
"psr/container-implementation": "1.0.0" "psr/container-implementation": "1.0.0"

2
lang/en.yml
View File

@ -111,6 +111,8 @@ en:
NEW: 'Add new record' NEW: 'Add new record'
NEXT: 'Go to next record' NEXT: 'Go to next record'
PREVIOUS: 'Go to previous record' PREVIOUS: 'Go to previous record'
EditPermissionsFailure: 'It seems you don''t have the necessary permissions to edit "{ObjectTitle}"'
ViewPermissionsFailure: 'It seems you don''t have the necessary permissions to view "{ObjectTitle}"'
SilverStripe\Forms\GridField\GridFieldEditButton: SilverStripe\Forms\GridField\GridFieldEditButton:
EDIT: Edit EDIT: Edit
SilverStripe\Forms\GridField\GridFieldFilterHeader: SilverStripe\Forms\GridField\GridFieldFilterHeader:

2
phpcs.xml.dist
View File

@ -18,7 +18,7 @@
<exclude name="Generic.Files.LineLength.TooLong" /> <exclude name="Generic.Files.LineLength.TooLong" />
<exclude name="PEAR.Functions.ValidDefaultValue.NotAtEnd" /> <exclude name="PEAR.Functions.ValidDefaultValue.NotAtEnd" />
</rule> </rule>
<rule phpcbf-only="true" ref="Squiz.Strings.ConcatenationSpacing"> <rule ref="Squiz.Strings.ConcatenationSpacing">
<properties> <properties>
<property name="spacing" value="1" /> <property name="spacing" value="1" />
<property name="ignoreNewlines" value="true"/> <property name="ignoreNewlines" value="true"/>

20
src/Forms/GridField/GridFieldDetailForm_ItemRequest.php
View File

@ -111,7 +111,11 @@ class GridFieldDetailForm_ItemRequest extends RequestHandler
public function view($request) public function view($request)
{ {
if (!$this->record->canView()) { if (!$this->record->canView()) {
$this->httpError(403); $this->httpError(403, _t(
__CLASS__ . '.ViewPermissionsFailure',
'It seems you don\'t have the necessary permissions to view "{ObjectTitle}"',
['ObjectTitle' => $this->record->singular_name()]
));
} }
$controller = $this->getToplevelController(); $controller = $this->getToplevelController();
@ -189,8 +193,11 @@ class GridFieldDetailForm_ItemRequest extends RequestHandler
if (!$this->record->canView()) { if (!$this->record->canView()) {
$controller = $this->getToplevelController(); $controller = $this->getToplevelController();
// TODO More friendly error return $controller->httpError(403, _t(
return $controller->httpError(403); __CLASS__ . '.ViewPermissionsFailure',
'It seems you don\'t have the necessary permissions to view "{ObjectTitle}"',
['ObjectTitle' => $this->record->singular_name()]
));
} }
$fields = $this->component->getFields(); $fields = $this->component->getFields();
@ -490,7 +497,12 @@ class GridFieldDetailForm_ItemRequest extends RequestHandler
// Check permission // Check permission
if (!$this->record->canEdit()) { if (!$this->record->canEdit()) {
return $this->httpError(403); $this->httpError(403, _t(
__CLASS__ . '.EditPermissionsFailure',
'It seems you don\'t have the necessary permissions to edit "{ObjectTitle}"',
['ObjectTitle' => $this->record->singular_name()]
));
return null;
} }
// Save from form data // Save from form data

4
src/Security/PasswordExpirationMiddleware.php
View File

@ -25,13 +25,13 @@ class PasswordExpirationMiddleware implements HTTPMiddleware
/** /**
* Session key for persisting URL of the password change form * Session key for persisting URL of the password change form
*/ */
const SESSION_KEY_REDIRECT = __CLASS__.'.change password redirect'; const SESSION_KEY_REDIRECT = __CLASS__ . '.change password redirect';
/** /**
* Session key for persisting a flag allowing to process the current request * Session key for persisting a flag allowing to process the current request
* without performing password expiration check * without performing password expiration check
*/ */
const SESSION_KEY_ALLOW_CURRENT_REQUEST = __CLASS__.'.allow current request'; const SESSION_KEY_ALLOW_CURRENT_REQUEST = __CLASS__ . '.allow current request';
/** /**
* List of URL patterns allowed for users to visit where * List of URL patterns allowed for users to visit where