Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
There are a number of inconsistent declarations in js files. For example these are a few examples:
Attaches behavior for the Editor module.
Attaches behaviors for the Contextual module
Attaches the behaviors for the Color module.
Attaches the behavior for drupalAnnouce.
Javascript behaviors for the Book module.
Defines Javascript behaviors for the block_content module.
Proposed resolution
Maybe come up with some kind of "standardization", if that is possible.
I submit a patch with my proposal.
Remaining tasks
Postponed on an
Conform to Documenting behaviors.
User interface changes
None.
API changes
None
Data model changes
None
Comment | File | Size | Author |
---|---|---|---|
#2 | inconsistent_comment_declarations-2820494-2.patch | 5.76 KB | vagelis-prokopiou |
Issue fork drupal-2820494
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
vagelis-prokopiou CreditAttribution: vagelis-prokopiou as a volunteer commentedHere is the patch. My suggestion is:
Defines JavaScript behaviors for the...
Comment #3
cilefen CreditAttribution: cilefen as a volunteer commentedThis is probably postponed on your other "JavaScript" patch.
Comment #4
vagelis-prokopiou CreditAttribution: vagelis-prokopiou as a volunteer commentedYes, sure. Do what you got to do.
Comment #5
cilefen CreditAttribution: cilefen as a volunteer commentedRelate this to the other issue and mark it postponed.
Comment #6
vagelis-prokopiou CreditAttribution: vagelis-prokopiou as a volunteer commentedComment #7
vagelis-prokopiou CreditAttribution: vagelis-prokopiou as a volunteer commentedComment #17
quietone CreditAttribution: quietone as a volunteer commentedThis is no longer postponed.
The documentation is inconsistent but it is also correct, so I don't think this is a bug. Changing to task.
If this change is made how will this consistency be maintained? I am all for consistency but not sure about this one. I'll leave it to those who work with the JavaScript files, adding JavaScript tag.
Comment #21
quietone CreditAttribution: quietone at PreviousNext commentedThere is an existing standard for Documenting behaviors. The patch here is not following that so a new patch will be needed.
Adding tags and parent.
Comment #24
apadernoComment #25
apaderno