Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sillygwailo’s picture

Assigned: Unassigned » sillygwailo

Taking this with baldwinlouie.

sillygwailo’s picture

Here's a patch.

sillygwailo’s picture

Status: Active » Needs review
dawehner’s picture

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.phpundefined
@@ -1161,12 +1161,12 @@ public function validateView(array $form, array &$form_state) {
+   * Implements Drupal\views\Plugin\views\wizard\WizardInterface::createView().

The standard is {@inheritdoc}, just have a look at [#1354]

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.phpundefined
@@ -1161,12 +1161,12 @@ public function validateView(array $form, array &$form_state) {
+      throw new WizardException('Attempted to createView with values that have not been validated.');

I guess if we already change this exception message something like "Attempted to create a view with values that have not been validated."

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardInterface.phpundefined
@@ -55,6 +55,6 @@ public function validateView(array $form, array &$form_state);
+  function createView(array $form, array &$form_state);

This should be a public method

Status: Needs review » Needs work

The last submitted patch, 2002434-rename-create_view.patch, failed testing.

sillygwailo’s picture

Status: Needs work » Needs review
FileSize
3.57 KB

Re-rolled with the feedback.

Status: Needs review » Needs work
Issue tags: -Novice, -VDC

The last submitted patch, 2002434-rename-create_view.patch, failed testing.

dawehner’s picture

Status: Needs work » Needs review
Issue tags: +Novice, +VDC

#2: 2002434-rename-create_view.patch queued for re-testing.

dawehner’s picture

#6: 2002434-rename-create_view.patch queued for re-testing.

oenie’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me now.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Needs a re-roll

 curl http://drupal.org/files/2002434-rename-create_view_0.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3652  100  3652    0     0   9835      0 --:--:-- --:--:-- --:--:-- 18728
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.php:1161
error: core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.php: patch does not apply
sillygwailo’s picture

Status: Needs work » Needs review
FileSize
3.56 KB

Re-rolled against a recent pull of 8.x (retrieve_validated_view in the original patch changed in core to retrieveValidatedView so it should match in the 'unchanged' lines now).

dawehner’s picture

Status: Needs review » Needs work
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.phpundefined
@@ -1161,12 +1161,12 @@ public function validateView(array $form, array &$form_state) {
+   * Implements Drupal\views\Plugin\views\wizard\WizardInterface::createView().
...
+  function createView(array $form, array &$form_state) {

public and @inheritdoc

sillygwailo’s picture

Status: Needs work » Needs review
FileSize
3.51 KB

Added, and re-rolled.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community
+++ b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.phpundefined
@@ -198,7 +198,7 @@ public function getSorts() {
   /**
-   * Implements Drupal\views\Plugin\views\wizard\WizardInterface::build_form().
+   * {@inheritdoc}
    */
   function build_form(array $form, array &$form_state) {

Even I don't like this unrelated change ...

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

If even @dawehner does not like the unrelated change displayed in #16... then lets not do it :)

sillygwailo’s picture

Status: Needs work » Needs review
FileSize
3.2 KB

Re-rolled with inheritDoc only for createView().

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Thank you!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 577949f and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.