implement category sending to nosto #1102
Annotations
2 errors and 26 warnings
Code Sniffer:
Model/Service/Sync/Upsert/SyncService.php#L253
Use of die language construct is discouraged.
|
Code Sniffer
Process completed with exit code 1.
|
Mess Detect
Your workflow is using a version of actions/cache that is scheduled for deprecation, actions/cache@v1. Please update your workflow to use either v3 or v4 of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
|
Code Sniffer
Your workflow is using a version of actions/cache that is scheduled for deprecation, actions/cache@v1. Please update your workflow to use either v3 or v4 of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
|
Package
Your workflow is using a version of actions/cache that is scheduled for deprecation, actions/cache@v1. Please update your workflow to use either v3 or v4 of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
|
Mess Detect:
Block/Adminhtml/Form/Field/Tokens.php#L87
Avoid variables with short names like $id. Configured minimum length is 3.
|
Mess Detect:
Block/Product.php#L63
The class Product has a coupling between objects value of 18. Consider to reduce the number of dependencies under 13.
|
Mess Detect:
Block/TaggingTrait.php#L68
The method _toHtml is not named in camelCase.
|
Mess Detect:
Console/Command/NostoAccountConnectCommand.php#L142
Avoid variables with short names like $io. Configured minimum length is 3.
|
Mess Detect:
Console/Command/NostoAccountConnectCommand.php#L174
Avoid variables with short names like $io. Configured minimum length is 3.
|
Mess Detect:
Console/Command/NostoAccountConnectCommand.php#L214
The method getTokensFromInput() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.
|
Mess Detect:
Console/Command/NostoAccountConnectCommand.php#L214
The method getTokensFromInput() has an NPath complexity of 576. The configured NPath complexity threshold is 200.
|
Mess Detect:
Console/Command/NostoAccountConnectCommand.php#L214
Avoid variables with short names like $io. Configured minimum length is 3.
|
Mess Detect:
Console/Command/NostoAccountRemoveCommand.php#L122
Avoid variables with short names like $io. Configured minimum length is 3.
|
Mess Detect:
Console/Command/NostoAccountRemoveCommand.php#L144
Avoid variables with short names like $io. Configured minimum length is 3.
|
Mess Detect
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Package
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Code Sniffer:
Model/Category/Repository.php#L132
Possible slow SQL method distinct detected
|
Code Sniffer:
Model/Category/Repository.php#L189
The closing brace for the class must go on the next line after the body
|
Code Sniffer:
Model/Service/Update/CategoryUpdateService.php#L135
Comment refers to a TODO task "@ugljesa Check instanceof"
|
Code Sniffer:
Model/Service/Sync/AbstractBulkPublisher.php#L131
Comment refers to a TODO task "@ugljesa Check the code"
|
Code Sniffer:
Model/Service/Sync/Upsert/Category/AsyncBulkConsumer.php#L102
Comment refers to a TODO task "Add store"
|
Code Sniffer:
Model/Service/Sync/Upsert/SyncService.php#L223
Comment refers to a TODO task "send to nosto customer"
|
Code Sniffer:
Model/Service/Sync/Upsert/SyncService.php#L235
Line exceeds 120 characters; contains 150 characters
|
Code Sniffer:
Model/Service/Sync/Upsert/SyncService.php#L253
Use of die language construct is discouraged.
|
Code Sniffer:
Model/Service/Sync/Upsert/SyncService.php#L257
Function closing brace must go on the next line following the body; found 1 blank lines before brace
|
Code Sniffer:
Model/Service/Sync/AbstractBulkConsumer.php#L47
Comment refers to a TODO task "needs to be abstracted to include categories as well"
|
Code Sniffer
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
composer-zip-archive
|
993 KB |
|
phpcs-xml-result
|
1.04 KB |
|
phpmd-xml-result
|
4.09 KB |
|