Magento Coding Standard (PHPCS) report for nowpost/module-click-collect ======================================================================= Package version: 0.0.4 | Magento: 2.4.9 | Tested: 2026-06-01 12 errors, 112 warnings in 14 files Plugin/Checkout/ValidationPlugin.php 59:8 warning Line exceeds 120 characters; contains 129 characters (Generic.Files.LineLength.TooLong) 69:12 warning $args parameter is missing in method annotation (Magento2.Annotation.MethodArguments.ArgumentMissing) Helper/PhoneNormalizer.php 46:155 warning Line exceeds 120 characters; contains 155 characters (Generic.Files.LineLength.TooLong) 52:129 warning Line exceeds 120 characters; contains 129 characters (Generic.Files.LineLength.TooLong) 64:208 warning Line exceeds 120 characters; contains 208 characters (Generic.Files.LineLength.TooLong) 71:139 warning Line exceeds 120 characters; contains 139 characters (Generic.Files.LineLength.TooLong) 74:173 warning Line exceeds 120 characters; contains 173 characters (Generic.Files.LineLength.TooLong) dev/seed-products.php 10:16 warning The use of function dirname() is discouraged; use Magento\Framework\Filesystem\DriverInterface::getParentDirectory() instead (Magento2.Functions.DiscouragedFunction.DiscouragedWithAlternative) 11:1 error "require" statement detected. File manipulations are discouraged. Concatenating is forbidden. Variables inside are insecure. (Magento2.Security.IncludeFile.FoundIncludeFile) 13:36 warning Direct use of $_SERVER Superglobal detected. (Magento2.Security.Superglobal.SuperglobalUsageWarning) 19:3 warning Empty CATCH statement detected (Magento2.CodeAnalysis.EmptyBlock.DetectedCatch) 30:127 warning Line exceeds 120 characters; contains 127 characters (Generic.Files.LineLength.TooLong) 39:5 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 53:5 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 64:134 warning Line exceeds 120 characters; contains 134 characters (Generic.Files.LineLength.TooLong) 65:137 warning Line exceeds 120 characters; contains 137 characters (Generic.Files.LineLength.TooLong) 66:132 warning Line exceeds 120 characters; contains 132 characters (Generic.Files.LineLength.TooLong) 67:133 warning Line exceeds 120 characters; contains 133 characters (Generic.Files.LineLength.TooLong) 68:139 warning Line exceeds 120 characters; contains 139 characters (Generic.Files.LineLength.TooLong) 74:9 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 76:7 warning Empty CATCH statement detected (Magento2.CodeAnalysis.EmptyBlock.DetectedCatch) 92:9 warning The use of function file_exists() is discouraged (Magento2.Functions.DiscouragedFunction.Discouraged) 95:14 warning The use of function is_dir() is discouraged; use Magento\Framework\Filesystem\DriverInterface::isDirectory() instead (Magento2.Functions.DiscouragedFunction.DiscouragedWithAlternative) 96:13 warning The use of function mkdir() is discouraged; use Magento\Framework\Filesystem\DriverInterface::createDirectory() instead (Magento2.Functions.DiscouragedFunction.DiscouragedWithAlternative) 99:9 warning The use of function copy() is discouraged; use Magento\Framework\Filesystem\DriverInterface::copy() instead (Magento2.Functions.DiscouragedFunction.DiscouragedWithAlternative) 107:9 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 109:9 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 123:5 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 139:7 warning Line exceeds 120 characters; contains 326 characters (Generic.Files.LineLength.TooLong) 146:5 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 148:5 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 151:1 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 152:1 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) 153:1 error Use of echo language construct is discouraged. (Magento2.Security.LanguageConstruct.DirectOutput) Model/Config.php 30:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 35:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 40:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 45:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 50:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 55:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 60:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 65:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 70:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) 75:5 warning Visibility must be declared on all constants if your project supports PHP 7.1 or later (PSR12.Properties.ConstantVisibility.NotFound) Observer/SyncOrderToApi.php 27:38 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 28:47 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 29:39 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 30:38 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 31:29 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 32:37 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 33:38 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) Observer/ClearQuotePickupOnCartChange.php 33:27 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 35:12 warning Comment block is missing (Magento2.Annotation.MethodAnnotationStructure.NoCommentBlock) 126:13 warning Comment block is missing (Magento2.Annotation.MethodAnnotationStructure.NoCommentBlock) 156:13 warning Comment block is missing (Magento2.Annotation.MethodAnnotationStructure.NoCommentBlock) 170:13 warning The use of function file_put_contents() is discouraged (Magento2.Functions.DiscouragedFunction.Discouraged) Observer/SaveOrderPickupData.php 27:37 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 34:12 warning Comment block is missing (Magento2.Annotation.MethodAnnotationStructure.NoCommentBlock) 102:135 warning Line exceeds 120 characters; contains 135 characters (Generic.Files.LineLength.TooLong) 109:132 warning Line exceeds 120 characters; contains 132 characters (Generic.Files.LineLength.TooLong) 110:141 warning Line exceeds 120 characters; contains 141 characters (Generic.Files.LineLength.TooLong) 111:129 warning Line exceeds 120 characters; contains 129 characters (Generic.Files.LineLength.TooLong) 112:144 warning Line exceeds 120 characters; contains 144 characters (Generic.Files.LineLength.TooLong) 113:129 warning Line exceeds 120 characters; contains 129 characters (Generic.Files.LineLength.TooLong) 114:132 warning Line exceeds 120 characters; contains 132 characters (Generic.Files.LineLength.TooLong) Setup/Patch/Data/AddNowPostOrderAttributes.php 43:8 warning If the @inheritdoc not inline it shouldn’t have braces (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 119:8 warning If the @inheritdoc not inline it shouldn’t have braces (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 127:8 warning If the @inheritdoc not inline it shouldn’t have braces (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 135:8 warning If the @inheritdoc not inline it shouldn’t have braces (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) Controller/Ajax/UpdateFee.php 54:27 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 63:12 warning @param is not found for one or more params in method annotation (Magento2.Annotation.MethodArguments.ParamMissing) 194:13 warning Comment block is missing (Magento2.Annotation.MethodAnnotationStructure.NoCommentBlock) 209:13 warning The use of function file_put_contents() is discouraged (Magento2.Functions.DiscouragedFunction.Discouraged) view/frontend/web/template/cart/shipping-rates.html 1:1 warning Avoid using closing slash with void tags, this can lead to unexpected behavior - "" (Magento2.Html.HtmlClosingVoidTags.HtmlClosingVoidElements) view/adminhtml/templates/order/pickup-info.phtml 38:1 warning Line exceeds 120 characters; contains 158 characters (Generic.Files.LineLength.TooLong) 47:146 warning Line exceeds 120 characters; contains 146 characters (Generic.Files.LineLength.TooLong) 55:53 warning Using escapeHtmlAttr on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 56:48 warning Using escapeHtmlAttr on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 57:39 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 62:1 warning Line exceeds 120 characters; contains 168 characters (Generic.Files.LineLength.TooLong) 63:118 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 63:148 warning Line exceeds 120 characters; contains 152 characters (Generic.Files.LineLength.TooLong) 68:1 warning Line exceeds 120 characters; contains 167 characters (Generic.Files.LineLength.TooLong) 69:95 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 69:124 warning Line exceeds 120 characters; contains 128 characters (Generic.Files.LineLength.TooLong) 74:1 warning Line exceeds 120 characters; contains 157 characters (Generic.Files.LineLength.TooLong) 75:77 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 96:1 warning Line exceeds 120 characters; contains 156 characters (Generic.Files.LineLength.TooLong) 97:77 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 102:1 warning Line exceeds 120 characters; contains 159 characters (Generic.Files.LineLength.TooLong) 103:77 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 107:1 warning Line exceeds 120 characters; contains 161 characters (Generic.Files.LineLength.TooLong) 108:118 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 108:142 warning Line exceeds 120 characters; contains 146 characters (Generic.Files.LineLength.TooLong) 112:1 warning Line exceeds 120 characters; contains 155 characters (Generic.Files.LineLength.TooLong) 113:95 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) 113:120 warning Line exceeds 120 characters; contains 124 characters (Generic.Files.LineLength.TooLong) 118:1 warning Line exceeds 120 characters; contains 155 characters (Generic.Files.LineLength.TooLong) 119:77 warning Using escapeHtml on $block is deprecated. Please use equivalent method on $escaper (Magento2.Legacy.EscapeMethodsOnBlockClass.Found) Api/NowPostApiClient.php 28:27 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 29:29 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) 30:38 warning Missing PHP DocBlock for class property. (Magento2.Commenting.ClassPropertyPHPDocFormatting.Missing) Block/ConfigProvider.php 51:8 warning If the @inheritdoc not inline it shouldn’t have braces (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) Block/Adminhtml/Order/PickupInfo.php 66:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 67:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 74:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 75:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 82:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 83:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 90:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 91:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 98:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 99:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 106:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 107:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 114:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 115:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 122:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 123:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 130:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 131:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 138:5 warning Missing short description (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 139:8 warning There must be exactly one blank line before tags (Magento2.Annotation.MethodAnnotationStructure.MethodAnnotation) 192:13 warning Comment block is missing (Magento2.Annotation.MethodAnnotationStructure.NoCommentBlock)