Rename \Drupal\Core\Config\Entity\ConfigEntityInterface::getOriginalID() to \Drupal\Core\Config\Entity\ConfigEntityInterface::getOriginalId() and setOriginalID() to setOriginalId(). This change is consistent with our coding standards.

CommentFileSizeAuthor
#11 drupal_2113319_11.patch14.06 KBXano
#7 drupal_2113319_6.patch13.34 KBXano
#3 drupal_2113319_3.patch13.31 KBXano
#1 drupal_2113319_1.patch13.31 KBXano
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Assigned: Xano » Unassigned
Status: Active » Needs review
FileSize
drupal_2113319_1.patch13.31 KB

Status: Needs review » Needs work

The last submitted patch, drupal_2113319_1.patch, failed testing.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review
FileSize
drupal_2113319_3.patch13.31 KB
tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Needs work 
+++ b/core/lib/Drupal/Core/Config/Entity/ConfigStorageController.php
@@ -432,7 +432,7 @@ public function save(EntityInterface $entity) {
-      $entity->setOriginalID($entity->id());
+      $entity->setOriginalId$entity->id());

This (also everywhere else) is missing the "(" part.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Assigned: Unassigned » Xano
tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Assigned: Xano » Unassigned

Oops that was a crosspost, I meant the first patch.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review
FileSize
drupal_2113319_6.patch13.34 KB

Speaking of crossposts ;)

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented

I'm not seeing any function definitions in the patch?!

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

The method names were apparently fixed a while ago, but some erroneous usages remained.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Needs work

I found another instance in line 36 of \Drupal\views_ui\Tests\ViewUIObjectTest

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review
FileSize
drupal_2113319_11.patch14.06 KB

Ah, an array key. That's why PHPStorm did not find it.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Awesome, let's do it.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

#11: drupal_2113319_11.patch queued for re-testing.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, drupal_2113319_11.patch, failed testing.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review

#11: drupal_2113319_11.patch queued for re-testing.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community
Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

11: drupal_2113319_11.patch queued for re-testing.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

11: drupal_2113319_11.patch queued for re-testing.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Since PHP functions aren't case-sensitive, I don't believe this needs a change notice.

Status: Fixed » Closed (fixed)

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