In core/modules/views_ui/src/ViewUI.php there are several lines like this:

 array('data' => array('#type' => 'inline_template', '#template' => "<strong>{% trans 'Query' %}</strong>")),

However, elsewhere in core, each of #something elements is on a separate line, like this one from core/modules/system/tests/modules/theme_test/src/ThemeTestController.php:

    $element['test'] = array(
      '#type' => 'inline_template',
      '#template' => 'test-with-context {{ llama }}',
      '#context' => array('llama' => 'muuh'),
    );

Your mission, should you choose to accept it, is to re-format these lines so they match our lovely Coding Standards!

CommentFileSizeAuthor
#6 fix-whitespaces-of-inline-template-2409391-6.patch4.1 KBgbisht
#2 fix-whitespaces-of-inline-template-2409391-2.patch4.08 KBCogax
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Cogax’s picture

Cogax CreditAttribution: Cogax commented
Assigned: Unassigned » Cogax
Cogax’s picture

Cogax CreditAttribution: Cogax commented
Status: Active » Needs review
FileSize
fix-whitespaces-of-inline-template-2409391-2.patch4.08 KB

I've made some changes. Hope it fits.

Cogax’s picture

Cogax CreditAttribution: Cogax commented
Assigned: Cogax » Unassigned
karengrey’s picture

karengrey CreditAttribution: karengrey commented
Status: Needs review » Reviewed & tested by the community

Code is now re-formatted and meets coding standards.
Good job!

gvso’s picture

gvso
Primary language Spanish
CreditAttribution: gvso commented
Status: Reviewed & tested by the community » Needs work 
+              array(
+                'data' => array(
+                  '#type' => 'inline_template',
+                  '#template' => "<strong>{% trans 'Query' %}</strong>"
+                )
+              ),

We should include comma at the end of every array item

Something like this

+              array(
+                'data' => array(
+                  '#type' => 'inline_template',
+                  '#template' => "<strong>{% trans 'Query' %}</strong>",
+                ),
+              ),
gbisht’s picture

gbisht CreditAttribution: gbisht commented
Status: Needs work » Needs review
Issue tags: +SprintWeekend2015
FileSize
fix-whitespaces-of-inline-template-2409391-6.patch4.1 KB

Formatting done!

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman commented
Status: Needs review » Reviewed & tested by the community

Patch applies clean, #6 addressed remaining lack of comma usage.

gvso’s picture

gvso
Primary language Spanish
CreditAttribution: gvso commented

RTBC +1 for #6

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: fix-whitespaces-of-inline-template-2409391-6.patch, failed testing.

Status: Needs work » Needs review

gvso queued 6: fix-whitespaces-of-inline-template-2409391-6.patch for re-testing.

gvso’s picture

gvso
Primary language Spanish
CreditAttribution: gvso commented
Status: Needs review » Reviewed & tested by the community

Wondering why it failed. Marking it as RTBC again!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: fix-whitespaces-of-inline-template-2409391-6.patch, failed testing.

Status: Needs work » Needs review

gvso queued 6: fix-whitespaces-of-inline-template-2409391-6.patch for re-testing.

gvso’s picture

gvso
Primary language Spanish
CreditAttribution: gvso commented
Status: Needs review » Reviewed & tested by the community

Why it failed again? It's weird. Marking as RTBC one more time :)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 610e90e and pushed to 8.0.x. Thanks!

  • alexpott committed 610e90e on 8.0.x 
    Issue #2409391 by Cogax, gulab.bisht: Fix the whitespace of...

Status: Fixed » Closed (fixed)

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