Follow up for #1648930-250: Introduce configuration schema and use for translation 14.
Namespace docs standard fix in core/modules/locale/lib/Drupal/locale/StringStorageInterface.php

Problem/Motivation

Originating issue had changes to this file, but only \namespace style fixes. It was cluttering the issue, and needed to be a follow-up.

Proposed resolution

Go through StringStorageInterface.php and fix it to be correct according to http://drupal.org/node/1354#namespaces

Remaining tasks

  • make initial patch of changes, keep focused on only docs and style updates. no refactoring. if refactoring is needed... make it a separate follow-up. :)
  • get review
  • iterate. patch, review.

Test bot coverage should be sufficient. No manual testing or screenshots needed.

User interface changes

no ui changes.

API changes

no api changes.

Files: 
CommentFileSizeAuthor
#15 StringStorageInterface_docfixes-1852288-15.patch5.07 KBunderq
PASSED: [[SimpleTest]]: [MySQL] 49,559 pass(es). View
#13 StringStorageInterface_docfixes-1852288-13.patch4.92 KBunderq
PASSED: [[SimpleTest]]: [MySQL] 49,192 pass(es). View
#8 StringStorageInterface_docfixes-1852288-8.patch4.92 KBunderq
PASSED: [[SimpleTest]]: [MySQL] 49,220 pass(es). View
#1 StringStorageInterface_docfixes-1852288-1.patch3.67 KBaaronott
FAILED: [[SimpleTest]]: [MySQL] 48,682 pass(es), 2 fail(s), and 3 exception(s). View

Comments

aaronott’s picture

FileSize
3.67 KB
FAILED: [[SimpleTest]]: [MySQL] 48,682 pass(es), 2 fail(s), and 3 exception(s). View
aaronott’s picture

Status: Active » Needs review

Status: Needs review » Needs work
Issue tags: -code cleanup, -Novice, -Configuration system, -D8MI, -language-config, -language-config-follow-up

The last submitted patch, StringStorageInterface_docfixes-1852288-1.patch, failed testing.

aaronott’s picture

Status: Needs work » Needs review
Issue tags: +code cleanup, +Novice, +Configuration system, +D8MI, +language-config, +language-config-follow-up
Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

YesCT’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +code cleanup, +Novice, +Configuration system, +D8MI, +language-config, +language-config-follow-up

The last submitted patch, StringStorageInterface_docfixes-1852288-1.patch, failed testing.

underq’s picture

FileSize
4.92 KB
PASSED: [[SimpleTest]]: [MySQL] 49,220 pass(es). View

Update patch :)

underq’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, StringStorageInterface_docfixes-1852288-8.patch, failed testing.

YesCT’s picture

+++ b/core/modules/locale/lib/Drupal/locale/StringStorageInterface.phpundefined
@@ -2,7 +2,7 @@
+ * Contains Drupal\locale\StringStorageInterface.

this should probably be, if this needs to be rerolled again, \Drupal\locale\...
patches are going in without it though.

Drupal\system\Tests\Upgrade\BlockUpgradePathTest fail probably unrelated.

YesCT’s picture

Status: Needs work » Needs review
underq’s picture

FileSize
4.92 KB
PASSED: [[SimpleTest]]: [MySQL] 49,192 pass(es). View

Reroll as suggested by YesCT :)

heyrocker’s picture

Status: Needs review » Needs work
+++ b/core/modules/locale/lib/Drupal/locale/StringStorageInterface.phpundefined
@@ -47,15 +45,15 @@ public function getStrings(array $conditions = array(), array $options = array()
-   *   Array of Drupal\locale\StringInterface objects matching the conditions.
-  */
+   *   Array of translation objects matching the conditions.

It appears we don't have a standard for this, but I actually prefer having the fully qualified object name in the description. Even if we don't make that change, the same object is called a "translation object" here and a "string object" earlier in the patch. It should be consistent.

Otherwise this looks good. Thanks!

underq’s picture

Status: Needs work » Needs review
FileSize
5.07 KB
PASSED: [[SimpleTest]]: [MySQL] 49,559 pass(es). View

I replaced all objects (string, location, translation) with a full qualified object name \Drupal\locale\StringInterface.

I hope this will by ok with you :)

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good :) Should not fail since only comment changes.

webchick’s picture

Component: configuration system » documentation
Assigned: Unassigned » jhodgdon

This is a docs thing, so moving to that component so Jennifer can have a look.

jhodgdon’s picture

Looks fine, I'll get it committed shortly. Thanks!

jhodgdon’s picture

Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Fixed

Committed to 8.x -- thanks again!

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