part of #2011290: [meta] standards cleanup to get ready for getting into core
Clean up the following file:
http://drupalcode.org/project/config_translation.git/blob/HEAD:/config_t...

Proposed resolution

Remaining tasks

User interface changes

No

API changes

No

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

robertdbailey’s picture

Title: config_translation.module standards cleanup to get ready for getting into core » config_translation.services.yml standards cleanup to get ready for getting into core
robertdbailey’s picture

Status: Needs work » Active
YesCT’s picture

Status: Active » Needs work

#1938580: [META] Make active config save format match the default yml file (order and quotes) might be of use while looking at this one.

[edit: no, this isn't config or schema...]

Need to find another example of services.yml in core.

YesCT’s picture

Status: Needs work » Active
YesCT’s picture

Issue summary: View changes

updated UI/API changes sections to include "no"

YesCT’s picture

YesCT’s picture

Priority: Normal » Minor

In views:
http://drupalcode.org/project/drupal.git/blob/HEAD:/core/modules/views/v...

   views.route_access_check:
     class: Drupal\views\ViewsAccessCheck
     tags:
       - { name: 'access_check' }

the name is in single quotes and there is a space before the }

in core:
http://drupalcode.org/project/drupal.git/blob/HEAD:/core/core.services.y...

   access_check.default:
     class: Drupal\Core\Access\DefaultAccessCheck
     tags:
       - { name: access_check }

We have:

services:
  config_translation.subscriber:
    class: Drupal\config_translation\Routing\RouteSubscriber
    tags:
      - { name: event_subscriber}
  config_translation.access_check:
    class: Drupal\config_translation\Access\ConfigNameCheck
    tags:
      - { name: access_check}

So, I dont see anything blocking here.
I'm not even sure if there is anything to change.
Are these services.yml files hand written, or saved?

YesCT’s picture

https://drupal.org/node/1937056
might be a change notice that says how to write it.

Oh, I should use blame and see which issue added this file.
#1985880: Convert to routing system
which references the WSCII conversion guide: https://drupal.org/node/1953342

I guess someone could add a space before the end }'s

Aside from that. It's fine as far as I can tell.

robertdbailey’s picture

Status: Active » Needs review
FileSize
560 bytes
YesCT’s picture

Status: Needs review » Reviewed & tested by the community

this look just fine! :) Thanks.

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Superb, thanks, committed.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

added file link to description