- Issue created by @samit.310@gmail.com
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 3:52am 10 March 2023 - Status changed to RTBC
over 1 year ago 5:25am 10 March 2023 - @hardikpandya opened merge request.
- Status changed to Needs work
over 1 year ago 2:07pm 10 March 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
/** - * Class ShortcutMenuSetCustomize. + * The ShortcutMenuSetCustomize Class.
What does the
phpcs
report say about that line?+/** + * The ShortcutMenuLazyBuilder Class. + */
What is wrong with those lines?
- protected $entity_type_manager; + protected $entityTypeManager;
That is not a necessary change, except in the case the code is already using
$entityTypeManager
. - First commit to issue fork.
- Merge request !23347117: Fix more phpcs and include #5 comments as well → (Open) created by kkalashnikov
- Status changed to Needs review
over 1 year ago 6:23am 14 March 2023 - Status changed to Needs work
over 1 year ago 8:40am 14 March 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
/** - * Class ShortcutMenuSetCustomize. + * The set customization for shortcut menu.
That does not describe what the class does, nor does it make sense.
+ /** + * Gets the Parent id from UUID. + */
Parameters and return value need to be documented.
- Assigned to elber
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 1:00pm 15 March 2023 - 🇧🇷Brazil elber Brazil
Hi I fixed the errors mentioned previously, please revise.
- Status changed to Needs work
3 months ago 7:48am 20 August 2024 Hi @elber,
Applied the latest changes made on MR!2, it was applied successfully but one file with 2 errors were still reported. Please see below:
➜ shortcut_menu git:(main) ✗ curl https://git.drupalcode.org/project/shortcut_menu/-/merge_requests/2.diff | patch -p1 % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 3125 0 3125 0 0 6552 0 --:--:-- --:--:-- --:--:-- 6706 patching file shortcut_menu.install patching file src/Form/ShortcutMenuSetCustomize.php patching file src/ShortcutMenuLazyBuilder.php ➜ shortcut_menu git:(main) ✗ .. ➜ contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig shortcut_menu FILE: /Users/PrometInterns/Demo-site/drupal-orgissue/web/modules/contrib/shortcut_menu/src/Form/ShortcutMenuSetCustomize.php ---------------------------------------------------------------------------------------------------------------------------- FOUND 2 ERRORS AFFECTING 2 LINES ---------------------------------------------------------------------------------------------------------------------------- 177 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null" 196 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null" ---------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY ---------------------------------------------------------------------------------------------------------------------------- Time: 266ms; Memory: 10MB
Kindly check
Thanks,
Jake- 🇮🇹Italy apaderno Brescia, 🇮🇹
avpaderno → changed the visibility of the branch 3347117-phpcs-fixes to hidden.
- Merge request !5Created a new merge request to get the list of all the PHP_CodeSniffer errors/warnings to fix → (Open) created by apaderno
- 🇮🇹Italy apaderno Brescia, 🇮🇹
avpaderno → changed the visibility of the branch 3347117-gitlab-ci-reports to hidden.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
- Assigned to apaderno
- Status changed to Needs review
3 months ago 7:38pm 20 August 2024 - Issue was unassigned.