nexusnovaz → changed the visibility of the branch 3549970-use-ignoredeprecations-instead to active.
nexusnovaz → changed the visibility of the branch 3549970-use-ignoredeprecations-instead to hidden.
This as a patch as well as the patch from https://www.drupal.org/project/text_with_title/issues/3522280#comment-16... 🐛 Fatal error when creating fields Active have worked for me in D11!
nexusnovaz → made their first commit to this issue’s fork.
nexusnovaz → made their first commit to this issue’s fork.
nexusnovaz → made their first commit to this issue’s fork.
I noticed a small error that occurred when i ran through the steps. I've added the ddev auth ssh step into the CONTRIBUTING.md document to note this. This will allow the ddev container to the use the personal ssh keys on the host machine, which is needed for the git clone commands. Please can someone review !745.
nexusnovaz → made their first commit to this issue’s fork.
Commenting on here as requested in the slack message.
nexusnovaz → made their first commit to this issue’s fork.
Commenting as requested. Its an interesting format to me. However, personally i prefer the video/voice meetings.
zoocha will → credited nexusnovaz → .
Hey, can MR !88 please be reviewed.
Thanks.
nexusnovaz → made their first commit to this issue’s fork.
Commenting per slack orders!
I've made a MR (!13005), so if someone could take a look at the text i've added there, that would be great.
nexusnovaz → made their first commit to this issue’s fork.
Moving this to needs review. It has passed (with warnings) on PHPUnit Tests for PHP 8.5.
I've updated the comment in MR !4439. Could this please be reviewed? I wasn't able to resolve the comments, so if someone else could, that would be great!
Thanks.
nexusnovaz → made their first commit to this issue’s fork.
I've made a change in the location of the comment and the comment itself. But once again, i know this needs to wait to be merged in! Will leave as Active for now.
Will move this back to NR as i've made the changed. Thanks @nicxvan for those, I wasn't sure if it would do more than 1 commit, so i did it manually.
nexusnovaz → made their first commit to this issue’s fork.
Ahh, thanks @nicxvan, i was wondering why an unrelated test was failing. I see they all passed the pipeline now, so ill move to NR
I've made a new commit, just need to do the tests which ill work on, but wont hate if someone gets to it before me. See this screenshot for format.
Hey, yeah, that makes sense. I've gone ahead and altered to be all on one line with it chaining calls. Please review again.
Thanks.
I've updated this, mind reviewing again?
I've opened MR !16 for this. But like i mentioned in #3539110-2: Wrong label for action button → should we change the titles?
This one appears to go a little deeper. For example, some of the pages have the title of "Add Edit UUID Configuration" or "Edit Edit UUID Configuration"
I'll do the change you requested while i wait to see if what i mentioned just above is in scope
nexusnovaz → made their first commit to this issue’s fork.
Made the quick change in MR !15. Please can someone review.
nexusnovaz → made their first commit to this issue’s fork.
Please can someone review MR !14.
Thanks
nexusnovaz → made their first commit to this issue’s fork.
Please can someone review MR !13. Just updated the docblock to hopefully be descriptive enough.
Thanks
nexusnovaz → made their first commit to this issue’s fork.
Thanks @catch, im still learning something new every day!
I've gone ahead and added that SelectInterface comment to the doc block. Ready for review.
Code looks good to me. Nice and simple.
I've added the statement and opened the MR 12755. However, i know this is dependent on 🐛 DefaultLazyPluginCollection::initializePlugin() throws the wrong sort of exception Active , however, im hoping that we can merge that in without any issues once upstream. I won't mark as needs review just yet, but will keep an eye on that parent ticket.
nexusnovaz → made their first commit to this issue’s fork.
Hey @joachim, I suspect @smustgrave was referring to using t() instead of sprintf(). Just in case he didn't mean this, i won't mark as needs work
I think I finally got this all working! I created a new function to separate it all out. Please review!
Yeah, i was struggling with it, coming back to it today. Thanks for noticing that spelling mistake!
I fear as though i have messed up the pulling of 11.x. Though i did make the change to remove the try catch clause and replacing it with a guard clause. Would someone be so kind as to explain what to do in this situation? For example, i did a git pull origin 11.x then git config pull.rebase false once it gave me some warning, then the git pull once again. Sorted the conflict (or so i thought) and pushed.
nexusnovaz → made their first commit to this issue’s fork.
Hey @ankitv18, for my learning, whats the need for those composer changes and the rift.info.yml for implementing ci/cd?
Hey, could you please review MR !100. I've added some text on the constant to mention that it updates the existing alias.
nexusnovaz → made their first commit to this issue’s fork.
PR LGTM, couldn't see any other references to QueryConditionInterface. I also couldn't find any direct replacements for QueryConditionInterface::having()
I was able to get tests working locally and have fixed the issue and the pipeline now passes. Happy to move this back to NR. Please review MR !12710
I'm moving this back to needs work. I'm currently struggling to get tests working locally, so if someone would like to take this on, feel free.
I've created MR !12710. Instead of the format listed, I believed it should be an Oxford comma, though this wouldn't be difficult to replace if not. Please could someone review.
nexusnovaz → made their first commit to this issue’s fork.
I've updated both branches to hopefully pass review bot. Please can we review?
nexusnovaz → made their first commit to this issue’s fork.
Please could you review MR !10. In theory it should also be compatible with 2.x branch.
Thanks
I have just opened 📌 Implement CI/CD for this module Active which i believe would also solve some issues. Requiring the pipeline to be fixed before any branches get merged.
I can take a look at setting this up as it is still useful rather than waiting for the pipeline
nexusnovaz → created an issue.
I've added some descriptions, i think this is good for review.
Thanks.
nexusnovaz → made their first commit to this issue’s fork.
Please could someone review MR !9.
I found a couple of other instances after running cspell where it was using British English spelling (or just a mistake) so i resolved those as well.
Thanks
I'll take a look at this.
I've found and removed the duplicated html elements. Please review MR !21
nexusnovaz → made their first commit to this issue’s fork.
I was also able to replicate this and made a quick patch. Will turn this into a PR shortly. Hopefully I have understood the issue and fixed it according to what others need.
My votes:
- Will Huggins (zoocha-will)
- Marine Gandy (Mupsi)
I resolved the issue that the review bot raised however there is the nightwatch test failing. Unsure if its just glitching and a rerun is fine, or if its because we're behind on commits?
nexusnovaz → made their first commit to this issue’s fork.
MR looks good to me. Passes PHPCS tests. Marking as RTBC.
Nice, thanks for that @avpaderno. I've made those changes and i believe it is all correct. Please can someone review MR !11912
Please review MR !20. Added the .cspell-project-words.txt file and also altered other files where needed
nexusnovaz → made their first commit to this issue’s fork.
I've made the changes needed for phpcs in MR !19. There is one remaining issue which mentions that
use Drupal\Console\Annotations\DrupalCommand;
is an unused use statement in src/Command/ExportDrupalCommand.php. Just wanted to confirm thats good to remove.
I'll take a look at this
I've made the changes in MR !11912 though there I have a question for Line 118. Currently the correct usage in the html tag should be href=":foo", but then @foo is used as the placeholder for the text surrounded by the html tag. However, there is no substitution for this in the array. Should the @foo also be changed to use :foo as it would appear they should have the same output?
nexusnovaz → made their first commit to this issue’s fork.
MR is simple and LGTM
nexusnovaz → created an issue.
nexusnovaz → created an issue.
I've rolled patch #3360268-3: Improve javascript to not need unsafe-eval → to an MR. Please can MR !23 be reviewed
nexusnovaz → changed the visibility of the branch 3360268-improve-javascript-to to hidden.
nexusnovaz → made their first commit to this issue’s fork.
Hi @isa.bel
It seems there is still some more to go. I reccomend using phpcs in your local environment as you go to fix all the issues.
I'm going to attach a file to this (phpcs_output.txt), for ease but you can also see the output below:
FILE: /var/www/html/web/modules/custom/jstimer/jstimer.module
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 12 ERRORS AND 2 WARNINGS AFFECTING 10 LINES
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment (Drupal.Commenting.FileComment.Missing)
3 | WARNING | [x] Unused use statement (Drupal.Classes.UnusedUseStatement.UnusedUse)
3 | ERROR | [x] There must be one blank line after the last USE statement; 0 found; (PSR2.Namespaces.UseDeclaration.SpaceAfterLastUse)
12 | ERROR | [x] Expected 1 blank line before function; 0 found (Squiz.WhiteSpace.FunctionSpacing.Before)
31 | ERROR | [x] Doc comment short description must start with a capital letter (Drupal.Commenting.DocComment.ShortNotCapital)
33 | ERROR | [ ] Missing parameter type (Drupal.Commenting.FunctionComment.MissingParamType)
35 | ERROR | [x] Separate the @param and @return sections by a blank line. (Drupal.Commenting.DocComment.TagGroupSpacing)
36 | WARNING | [ ] Line exceeds 80 characters; contains 87 characters (Drupal.Files.LineLength.TooLong)
38 | ERROR | [x] Incorrect spacing between argument "$format_string" and equals sign; expected 1 but found 0 (Squiz.Functions.FunctionDeclarationArgumentSpacing.SpaceBeforeEquals)
38 | ERROR | [x] Incorrect spacing between default value and equals sign for argument "$format_string"; expected 1 but found 0 (Squiz.Functions.FunctionDeclarationArgumentSpacing.SpaceAfterEquals)
39 | ERROR | [x] Expected 1 space after "="; 6 found (Squiz.WhiteSpace.OperatorSpacing.SpacingAfter)
39 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
40 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
40 | ERROR | [x] There should be no white space before a closing ")" (Drupal.WhiteSpace.CloseBracketSpacing.ClosingWhitespace)
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/web/modules/custom/jstimer/modules/jst_clock/jst_clock.module
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 21 ERRORS AND 3 WARNINGS AFFECTING 19 LINES
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
11 | WARNING | [ ] Format should be "* Implements hook_foo().", "* Implements hook_foo_BAR_ID_bar() for xyz_bar().",, "* Implements hook_foo_BAR_ID_bar() for xyz-bar.html.twig.", "* Implements hook_foo_BAR_ID_bar() for
| | xyz-bar.tpl.php.", or "* Implements hook_foo_BAR_ID_bar() for block templates." (Drupal.Commenting.HookComment.HookCommentFormat)
12 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph (Drupal.Commenting.DocComment.ShortSingleLine)
16 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4 (Drupal.WhiteSpace.ScopeIndent.IncorrectExact)
18 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4 (Drupal.WhiteSpace.ScopeIndent.IncorrectExact)
18 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
19 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
20 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
21 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
24 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4 (Drupal.WhiteSpace.ScopeIndent.IncorrectExact)
24 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
28 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4 (Drupal.WhiteSpace.ScopeIndent.IncorrectExact)
28 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
32 | ERROR | [ ] The array declaration extends to column 134 (the limit is 120). The array content should be split up over multiple lines (Drupal.Arrays.Array.LongLineDeclaration)
32 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
33 | WARNING | [x] A comma should follow the last multiline array item. Found: ) (Drupal.Arrays.Array.CommaLastItem)
36 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4 (Drupal.WhiteSpace.ScopeIndent.IncorrectExact)
36 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
37 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
38 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
39 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
40 | ERROR | [x] Array indentation error, expected 6 spaces but found 8 (Drupal.Arrays.Array.ArrayIndentation)
43 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4 (Drupal.WhiteSpace.ScopeIndent.IncorrectExact)
45 | WARNING | [ ] Line exceeds 80 characters; contains 91 characters (Drupal.Files.LineLength.TooLong)
50 | ERROR | [x] Missing function doc comment (Drupal.Commenting.FunctionComment.Missing)
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 20 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/web/modules/custom/jstimer/modules/jst_timer/jst_timer.module
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 52 ERRORS AND 5 WARNINGS AFFECTING 55 LINES
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
5 | ERROR | [x] Function comment short description must start with exactly one space (Drupal.Commenting.DocComment.ShortStartSpace)
6 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph (Drupal.Commenting.DocComment.ShortSingleLine)
7 | ERROR | [x] There must be exactly one blank line after the file comment (Drupal.Commenting.FileComment.SpacingAfterComment)
9 | WARNING | [x] Unused use statement (Drupal.Classes.UnusedUseStatement.UnusedUse)
9 | ERROR | [x] When importing a class with "use", do not include a leading \ (Drupal.Classes.UseLeadingBackslash.SeparatorStart)
12 | WARNING | [ ] Format should be "* Implements hook_foo().", "* Implements hook_foo_BAR_ID_bar() for xyz_bar().",, "* Implements hook_foo_BAR_ID_bar() for xyz-bar.html.twig.", "* Implements hook_foo_BAR_ID_bar() for
| | xyz-bar.tpl.php.", or "* Implements hook_foo_BAR_ID_bar() for block templates." (Drupal.Commenting.HookComment.HookCommentFormat)
13 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph (Drupal.Commenting.DocComment.ShortSingleLine)
22 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
25 | WARNING | [x] A comma should follow the last multiline array item. Found: 1 (Drupal.Arrays.Array.CommaLastItem)
28 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
33 | ERROR | [x] Array indentation error, expected 4 spaces but found 6 (Drupal.Arrays.Array.ArrayIndentation)
34 | ERROR | [x] Array indentation error, expected 4 spaces but found 6 (Drupal.Arrays.Array.ArrayIndentation)
41 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
42 | ERROR | [x] Array indentation error, expected 8 spaces but found 9 (Drupal.Arrays.Array.ArrayIndentation)
43 | ERROR | [x] Array indentation error, expected 8 spaces but found 9 (Drupal.Arrays.Array.ArrayIndentation)
44 | ERROR | [x] Array indentation error, expected 8 spaces but found 9 (Drupal.Arrays.Array.ArrayIndentation)
45 | ERROR | [x] Array indentation error, expected 8 spaces but found 9 (Drupal.Arrays.Array.ArrayIndentation)
49 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
52 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
58 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
61 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
66 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
74 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
80 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
85 | WARNING | [ ] Avoid backslash escaping in translatable strings when possible, use '' quotes instead (Drupal.Semantics.FunctionT.BackslashDoubleQuote)
87 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
94 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
99 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
105 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
114 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
120 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
125 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
128 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
135 | ERROR | [x] Functions must not contain multiple empty lines in a row; found 2 empty lines (Squiz.WhiteSpace.SuperfluousWhitespace.EmptyLines)
138 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
144 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
150 | ERROR | [x] Functions must not contain multiple empty lines in a row; found 2 empty lines (Squiz.WhiteSpace.SuperfluousWhitespace.EmptyLines)
153 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
159 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
168 | WARNING | [ ] Line exceeds 80 characters; contains 91 characters (Drupal.Files.LineLength.TooLong)
172 | ERROR | [x] Missing function doc comment (Drupal.Commenting.FunctionComment.Missing)
200 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph (Drupal.Commenting.DocComment.ShortSingleLine)
201 | ERROR | [x] There must be exactly one blank line before the tags in a doc comment (Drupal.Commenting.DocComment.SpacingBeforeTags)
201 | ERROR | [ ] Return type missing for @return tag in function comment (Drupal.Commenting.FunctionComment.MissingReturnType)
204 | ERROR | [ ] All functions defined in a module file must be prefixed with the module's name, found "translate_replacements" but expected "jst_timer_translate_replacements"
| | (Drupal.NamingConventions.ValidFunctionName.InvalidPrefix)
205 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
213 | ERROR | [x] Expected 1 blank line after function; 2 found (Squiz.WhiteSpace.FunctionSpacing.After)
217 | ERROR | [x] Expected 1 space(s) before asterisk; 2 found (Drupal.Commenting.DocCommentAlignment.SpaceBeforeStar)
218 | ERROR | [x] Expected 1 space(s) before asterisk; 2 found (Drupal.Commenting.DocCommentAlignment.SpaceBeforeStar)
220 | ERROR | [x] Short array syntax must be used to define arrays (Drupal.Arrays.DisallowLongArraySyntax.Found)
226 | ERROR | [x] Expected 1 blank line after function; 2 found (Squiz.WhiteSpace.FunctionSpacing.After)
230 | ERROR | [x] Expected 1 space(s) before asterisk; 2 found (Drupal.Commenting.DocCommentAlignment.SpaceBeforeStar)
231 | ERROR | [x] Expected 1 space(s) before asterisk; 2 found (Drupal.Commenting.DocCommentAlignment.SpaceBeforeStar)
253 | ERROR | [x] Concat operator must be surrounded by a single space (Squiz.Strings.ConcatenationSpacing.PaddingFound)
255 | ERROR | [x] Concat operator must be surrounded by a single space (Squiz.Strings.ConcatenationSpacing.PaddingFound)
260 | ERROR | [x] Expected 1 space after "="; 2 found (Squiz.WhiteSpace.OperatorSpacing.SpacingAfter)
285 | ERROR | [x] Expected 1 newline at end of file; 2 found (Drupal.Files.EndFileNewline.TooMany)
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 49 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Time: 191ms; Memory: 6MB
PHP CODE SNIFFER REPORT SUMMARY
------------------------------------------------------------------------------------------------
FILE ERRORS WARNINGS
------------------------------------------------------------------------------------------------
/var/www/html/web/modules/custom/jstimer/jstimer.module 12 2
/var/www/html/web/modules/custom/jstimer/modules/jst_clock/jst_clock.module 21 3
/var/www/html/web/modules/custom/jstimer/modules/jst_timer/jst_timer.module 52 5
------------------------------------------------------------------------------------------------
A TOTAL OF 85 ERRORS AND 10 WARNINGS WERE FOUND IN 3 FILES
------------------------------------------------------------------------------------------------
PHPCBF CAN FIX 81 OF THESE SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------
PHP CODE SNIFFER VIOLATION SOURCE SUMMARY
-------------------------------------------------------------------------------
SOURCE COUNT
-------------------------------------------------------------------------------
[x] Drupal.Arrays.DisallowLongArraySyntax.Found 31
[x] Drupal.Arrays.Array.ArrayIndentation 13
[x] Drupal.WhiteSpace.ScopeIndent.IncorrectExact 6
[ ] Drupal.Commenting.DocComment.ShortSingleLine 4
[x] Drupal.Commenting.DocCommentAlignment.SpaceBeforeStar 4
[ ] Drupal.Files.LineLength.TooLong 3
[x] Drupal.Arrays.Array.CommaLastItem 2
[x] Drupal.Classes.UnusedUseStatement.UnusedUse 2
[x] Drupal.Commenting.FunctionComment.Missing 2
[ ] Drupal.Commenting.HookComment.HookCommentFormat 2
[x] Squiz.Strings.ConcatenationSpacing.PaddingFound 2
[x] Squiz.WhiteSpace.FunctionSpacing.After 2
[x] Squiz.WhiteSpace.OperatorSpacing.SpacingAfter 2
[x] Squiz.WhiteSpace.SuperfluousWhitespace.EmptyLines 2
[ ] Drupal.Arrays.Array.LongLineDeclaration 1
[x] Drupal.Classes.UseLeadingBackslash.SeparatorStart 1
[x] Drupal.Commenting.DocComment.ShortNotCapital 1
[x] Drupal.Commenting.DocComment.ShortStartSpace 1
[x] Drupal.Commenting.DocComment.SpacingBeforeTags 1
[x] Drupal.Commenting.DocComment.TagGroupSpacing 1
[x] Drupal.Commenting.FileComment.Missing 1
[x] Drupal.Commenting.FileComment.SpacingAfterComment 1
[ ] Drupal.Commenting.FunctionComment.MissingParamType 1
[ ] Drupal.Commenting.FunctionComment.MissingReturnType 1
[x] Drupal.Files.EndFileNewline.TooMany 1
[ ] Drupal.NamingConventions.ValidFunctionName.InvalidPrefix 1
[ ] Drupal.Semantics.FunctionT.BackslashDoubleQuote 1
[x] Drupal.WhiteSpace.CloseBracketSpacing.ClosingWhitespace 1
[x] PSR2.Namespaces.UseDeclaration.SpaceAfterLastUse 1
[x] Squiz.Functions.FunctionDeclarationArgumentSpacing.SpaceAfterEquals 1
[x] Squiz.Functions.FunctionDeclarationArgumentSpacing.SpaceBeforeEquals 1
[x] Squiz.WhiteSpace.FunctionSpacing.Before 1
-------------------------------------------------------------------------------
A TOTAL OF 95 SNIFF VIOLATIONS WERE FOUND IN 32 SOURCES
-------------------------------------------------------------------------------
PHPCBF CAN FIX THE 24 MARKED SOURCES AUTOMATICALLY (81 VIOLATIONS IN TOTAL)
-------------------------------------------------------------------------------
nexusnovaz → created an issue.
nexusnovaz → created an issue.
MR LGTM. Seems like a simple change and it shouldn't in theory affect anything else too much currently.
Hey @charlliequadros, i've made a slight modification as the code is actually JS and not PHP, but it appears to still work. Care to review/test again
The change is simple and LGTM. Seems the error is unrelated through im not able to rerun the job?
I just realised the ticket i mentioned is under the same module. Closing this one as duplicate.
This is a ticket created as a kind of duplicate for https://www.drupal.org/project/documentation/issues/3489514 📌 Suggestion for: (2827301) 3.4. Preparing to Install Active as the documentation page says to open a new ticket on this module and upload the file here.
nexusnovaz → created an issue.
This was a simple one to quickly do. Please review MR !16