Fix the issues reported by phpcs

FILE: /enable_disable_node_toggle/enable_disable_node_toggle.libraries.yml ---------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ---------------------------------------------------------------------------------------------- 12 | ERROR | [x] Expected 1 newline at end of file; 0 found ---------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY ---------------------------------------------------------------------------------------------- FILE: /enable_disable_node_toggle/enable_disable_node_toggle.routing.yml ---------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ---------------------------------------------------------------------------------------------- 8 | ERROR | [x] Expected 1 newline at end of file; 0 found ---------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY ---------------------------------------------------------------------------------------------- FILE: /enable_disable_node_toggle/src/Controller/ChangeNodeStatus.php ----------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE ----------------------------------------------------------------------------------------------- 25 | WARNING | Node::load calls should be avoided in classes, use dependency injection instead -----------------------------------------------------------------------------------------------

Comments & Activities

Issue created by @omkar_yewale
@omkar_yewale opened merge request.
Comment over 1 year ago →
🇮🇳India omkar_yewale Mumbai
Please review the patch and !2 MR.
Issue was unassigned.
Status changed to RTBC about 1 year ago4:48pm 2 May 2023
Comment about 1 year ago →
🇵🇭Philippines kenyoOwen
Hi omkar_yewale

I applied patch #3 to the “Enable/Disable Node Status Toggle Button” module against Version 1.0.x-dev and confirmed that the errors and warning are resolved. Please see the screenshots attached.

For your review.
Thank you.
Status changed to Needs work about 1 year ago2:46pm 20 May 2023
Comment about 1 year ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
+ /** + * Constructs a new ChangeNodeStatus. + *
The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.
Comment about 1 year ago →
🇮🇳India kbk1992 Hyderabad
bharath-kondeti → made their first commit to this issue’s fork.
Status changed to Needs review about 1 year ago3:13pm 20 May 2023
Comment about 1 year ago →
🇮🇳India kbk1992 Hyderabad
Addressed #5. Putting a namespace in comment is giving more than 80 characters issue.

Fix the issues reported by phpcs

Problem/Motivation

Steps to reproduce

Proposed resolution

Comments & Activities