diff --git a/core/lib/Drupal/Core/Menu/MenuLinkBase.php b/core/lib/Drupal/Core/Menu/MenuLinkBase.php index c0f05e0..326a775 100644 --- a/core/lib/Drupal/Core/Menu/MenuLinkBase.php +++ b/core/lib/Drupal/Core/Menu/MenuLinkBase.php @@ -141,7 +141,7 @@ public function getUrlObject($title_attribute = TRUE) { return new Url($this->pluginDefinition['route_name'], $this->pluginDefinition['route_parameters'], $options); } else { - return Url::unrouted($this->pluginDefinition['url'], $options);; + return Url::unrouted($this->pluginDefinition['url'], $options); } } diff --git a/core/modules/views_ui/src/ViewEditForm.php b/core/modules/views_ui/src/ViewEditForm.php index ed87d4d..4112cce 100644 --- a/core/modules/views_ui/src/ViewEditForm.php +++ b/core/modules/views_ui/src/ViewEditForm.php @@ -326,6 +326,8 @@ public function save(array $form, FormStateInterface $form_state) { $form_state->setRedirectUrl(Url::unrouted("base://$destination")); } + $view->save(); + drupal_set_message($this->t('The view %name has been saved.', array('%name' => $view->label()))); // Remove this view from cache so we can edit it properly. diff --git a/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php b/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php index 0fa610d..472dc64 100644 --- a/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php +++ b/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php @@ -38,14 +38,14 @@ class UnroutedUrlTest extends UnitTestCase { * * @var string */ - protected $unrouted_external = 'http://drupal.org'; + protected $unroutedExternal = 'http://drupal.org'; /** * An unrouted, internal URL to test. * * @var string */ - protected $unrouted_internal = 'base://robots.txt'; + protected $unroutedInternal = 'base://robots.txt'; /** * {@inheritdoc} @@ -72,8 +72,8 @@ protected function setUp() { */ public function testCreateFromPath() { $urls = [ - Url::unrouted($this->unrouted_external), - Url::unrouted($this->unrouted_internal) + Url::unrouted($this->unroutedExternal), + Url::unrouted($this->unroutedInternal) ]; $this->assertInstanceOf('Drupal\Core\Url', $urls[0]); @@ -123,8 +123,8 @@ public function testIsExternal(array $urls) { * @covers ::toString() */ public function testToString(array $urls) { - $this->assertSame($this->unrouted_external, $urls[0]->toString()); - $this->assertSame($this->unrouted_internal, $urls[1]->toString()); + $this->assertSame($this->unroutedExternal, $urls[0]->toString()); + $this->assertSame($this->unroutedInternal, $urls[1]->toString()); } /** @@ -136,13 +136,13 @@ public function testToString(array $urls) { */ public function testToArray(array $urls) { $expected = array( - 'path' => $this->unrouted_external, + 'path' => $this->unroutedExternal, 'options' => array('external' => TRUE), ); $this->assertSame($expected, $urls[0]->toArray()); $expected = array( - 'path' => $this->unrouted_internal, + 'path' => $this->unroutedInternal, 'options' => array(), ); $this->assertSame($expected, $urls[1]->toArray());