tonyair/silverstripe-dms
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-dms synced 2024-10-22 12:05:56 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #153 from creative-commoners/pulls/2.0/remove-page-reference

Browse Source 
FIX Remove "Page" literal field when editing a doc set in a page context. Simplify exception test.
This commit is contained in:
sachajudd 2017-06-06 14:05:29 +12:00 committed by GitHub
commit 46aafb1df6
2 changed files with 25 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
code/model/DMSDocumentSet.php
View File

@ -117,8 +117,12 @@ class DMSDocumentSet extends DataObject
$gridFieldConfig->addComponent($sortableComponent);
}
$field = $fields->fieldByName('Root.Main.PageID');
$field->setTitle(_t('DMSDocumentSet.SHOWONPAGE', 'Show on page'));
// Don't show which page this is if we're already editing within a page context
if (Controller::curr() instanceof CMSPageEditController) {
$fields->removeByName('PageID');
} else {
$fields->fieldByName('Root.Main.PageID')->setTitle(_t('DMSDocumentSet.SHOWONPAGE', 'Show on page'));
}
$gridFieldConfig->getComponentByType('GridFieldDataColumns')
->setDisplayFields($self->getDocumentDisplayFields())

25
tests/DMSDocumentSetTest.php
View File

@ -194,11 +194,24 @@ class DMSDocumentSetTest extends SapphireTest
*/
public function testExceptionOnNoTitleGiven()
{
$set = DMSDocumentSet::create(array('Title' => ''));
try {
$set->write();
} catch (ValidationException $e) {
throw $e;
}
DMSDocumentSet::create(array('Title' => ''))->write();
}
/**
* Ensure that when editing in a page context that the "page" field is removed, or is labelled "Show on page"
* otherwise
*/
public function testPageFieldRemovedWhenEditingInPageContext()
{
$set = $this->objFromFixture('DMSDocumentSet', 'ds1');
$fields = $set->getCMSFields();
$this->assertInstanceOf('DropdownField', $fields->fieldByName('Root.Main.PageID'));
$pageController = new CMSPageEditController;
$pageController->pushCurrent();
$fields = $set->getCMSFields();
$this->assertNull($fields->fieldByName('Root.Main.PageID'));
}
}