diff --git a/composer.json b/composer.json index 12b539b33..5166dd89a 100644 --- a/composer.json +++ b/composer.json @@ -96,7 +96,7 @@ "codeception/codeception": "^2.5.0", "phpstan/phpstan": "^0.7", "phpunit/phpunit": "^7.0", - "shopsys/coding-standards": "v7.0.0-beta2", + "shopsys/coding-standards": "v7.0.0-beta3", "shopsys/http-smoke-testing": "v7.0.0-beta2" }, "conflict": { diff --git a/composer.lock b/composer.lock index ea9383cef..06c7d4736 100644 --- a/composer.lock +++ b/composer.lock @@ -4,7 +4,7 @@ "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "This file is @generated automatically" ], - "content-hash": "11528a4e46054f0cbf0e8837f5b86da1", + "content-hash": "863b8b209c12a9d014f128e6d26911c2", "packages": [ { "name": "arvenil/ninja-mutex", @@ -9612,16 +9612,16 @@ }, { "name": "shopsys/coding-standards", - "version": "v7.0.0-beta2", + "version": "v7.0.0-beta3", "source": { "type": "git", "url": "https://github.com/shopsys/coding-standards.git", - "reference": "bb397e146931f7b697e871a3b5355e68a29f0a64" + "reference": "094c55d286e209998bd2808c1167cbb406e9a10b" }, "dist": { "type": "zip", - "url": "https://api.github.com/repos/shopsys/coding-standards/zipball/bb397e146931f7b697e871a3b5355e68a29f0a64", - "reference": "bb397e146931f7b697e871a3b5355e68a29f0a64", + "url": "https://api.github.com/repos/shopsys/coding-standards/zipball/094c55d286e209998bd2808c1167cbb406e9a10b", + "reference": "094c55d286e209998bd2808c1167cbb406e9a10b", "shasum": "" }, "require": { @@ -9662,7 +9662,7 @@ "code quality", "coding standards" ], - "time": "2018-08-23T09:12:51+00:00" + "time": "2018-11-15T09:31:59+00:00" }, { "name": "shopsys/http-smoke-testing", diff --git a/src/Shopsys/ShopBundle/Controller/Admin/CustomerController.php b/src/Shopsys/ShopBundle/Controller/Admin/CustomerController.php index a3472ca1b..844bf4134 100644 --- a/src/Shopsys/ShopBundle/Controller/Admin/CustomerController.php +++ b/src/Shopsys/ShopBundle/Controller/Admin/CustomerController.php @@ -111,6 +111,24 @@ class CustomerController extends BaseCustomerController */ protected $domainRouterFactory; + /** + * @param \Shopsys\FrameworkBundle\Model\Customer\UserDataFactoryInterface $userDataFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerListAdminFacade $customerListAdminFacade + * @param \Shopsys\ShopBundle\Model\Customer\CustomerFacade $customerFacade + * @param \Shopsys\FrameworkBundle\Model\AdminNavigation\BreadcrumbOverrider $breadcrumbOverrider + * @param \Shopsys\FrameworkBundle\Model\Administrator\AdministratorGridFacade $administratorGridFacade + * @param \Shopsys\FrameworkBundle\Component\Grid\GridFactory $gridFactory + * @param \Shopsys\FrameworkBundle\Component\Domain\AdminDomainTabsFacade $adminDomainTabsFacade + * @param \Shopsys\FrameworkBundle\Model\Order\OrderFacade $orderFacade + * @param \Shopsys\FrameworkBundle\Model\Security\LoginAsUserFacade $loginAsUserFacade + * @param \Shopsys\FrameworkBundle\Component\Router\DomainRouterFactory $domainRouterFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerDataFactoryInterface $customerDataFactory + * @param \Shopsys\ShopBundle\Model\Customer\BillingAddressFacade $billingAddressFacade + * @param \Shopsys\ShopBundle\Model\Customer\BillingAddressDataFactory $billingAddressDataFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\UserFactory $userFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\DeliveryAddressDataFactory $deliveryAddressDataFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\DeliveryAddressFactory $deliveryAddressFactory + */ public function __construct( UserDataFactoryInterface $userDataFactory, CustomerListAdminFacade $customerListAdminFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Admin/LoginController.php b/src/Shopsys/ShopBundle/Controller/Admin/LoginController.php index 45d23be45..953d68b34 100644 --- a/src/Shopsys/ShopBundle/Controller/Admin/LoginController.php +++ b/src/Shopsys/ShopBundle/Controller/Admin/LoginController.php @@ -12,6 +12,9 @@ class LoginController extends BaseLoginController { + /** + * @param \Shopsys\FrameworkBundle\Component\Router\DomainRouter $domainRouter + */ private function changeDomainContext(DomainRouter $domainRouter) { $mainAdminDomainHost = $domainRouter->getContext()->getHost(); diff --git a/src/Shopsys/ShopBundle/Controller/Front/AdvertController.php b/src/Shopsys/ShopBundle/Controller/Front/AdvertController.php index 848d6b6cf..6bbe093a4 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/AdvertController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/AdvertController.php @@ -11,6 +11,9 @@ class AdvertController extends FrontBaseController */ private $advertFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Advert\AdvertFacade $advertFacade + */ public function __construct(AdvertFacade $advertFacade) { $this->advertFacade = $advertFacade; diff --git a/src/Shopsys/ShopBundle/Controller/Front/ArticleController.php b/src/Shopsys/ShopBundle/Controller/Front/ArticleController.php index a3552538e..4c53bad96 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/ArticleController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/ArticleController.php @@ -12,6 +12,9 @@ class ArticleController extends FrontBaseController */ private $articleFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Article\ArticleFacade $articleFacade + */ public function __construct(ArticleFacade $articleFacade) { $this->articleFacade = $articleFacade; diff --git a/src/Shopsys/ShopBundle/Controller/Front/BestsellingProductController.php b/src/Shopsys/ShopBundle/Controller/Front/BestsellingProductController.php index 5c8aa7a22..1327b3f47 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/BestsellingProductController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/BestsellingProductController.php @@ -25,6 +25,11 @@ class BestsellingProductController extends FrontBaseController */ private $currentCustomer; + /** + * @param \Shopsys\FrameworkBundle\Model\Product\BestsellingProduct\CachedBestsellingProductFacade $cachedBestsellingProductFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Customer\CurrentCustomer $currentCustomer + */ public function __construct( CachedBestsellingProductFacade $cachedBestsellingProductFacade, Domain $domain, @@ -35,6 +40,9 @@ public function __construct( $this->currentCustomer = $currentCustomer; } + /** + * @param \Shopsys\FrameworkBundle\Model\Category\Category $category + */ public function listAction(Category $category) { $bestsellingProducts = $this->cachedBestsellingProductFacade->getAllOfferedBestsellingProducts( diff --git a/src/Shopsys/ShopBundle/Controller/Front/BrandController.php b/src/Shopsys/ShopBundle/Controller/Front/BrandController.php index add71852f..a3833835e 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/BrandController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/BrandController.php @@ -11,6 +11,9 @@ class BrandController extends FrontBaseController */ private $brandFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Product\Brand\BrandFacade $brandFacade + */ public function __construct( BrandFacade $brandFacade ) { diff --git a/src/Shopsys/ShopBundle/Controller/Front/BreadcrumbController.php b/src/Shopsys/ShopBundle/Controller/Front/BreadcrumbController.php index d4131885e..0e267942e 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/BreadcrumbController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/BreadcrumbController.php @@ -11,6 +11,9 @@ class BreadcrumbController extends FrontBaseController */ private $breadcrumbResolver; + /** + * @param \Shopsys\FrameworkBundle\Component\Breadcrumb\BreadcrumbResolver $breadcrumbResolver + */ public function __construct( BreadcrumbResolver $breadcrumbResolver ) { diff --git a/src/Shopsys/ShopBundle/Controller/Front/CartController.php b/src/Shopsys/ShopBundle/Controller/Front/CartController.php index 39e13b2e6..be84188d6 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/CartController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/CartController.php @@ -58,6 +58,15 @@ class CartController extends FrontBaseController */ private $errorService; + /** + * @param \Shopsys\FrameworkBundle\Model\Product\Accessory\ProductAccessoryFacade $productAccessoryFacade + * @param \Shopsys\FrameworkBundle\Model\Cart\CartFacade $cartFacade + * @param \Shopsys\FrameworkBundle\Model\Customer\CurrentCustomer $currentCustomer + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\TransportAndPayment\FreeTransportAndPaymentFacade $freeTransportAndPaymentFacade + * @param \Shopsys\FrameworkBundle\Model\Order\Preview\OrderPreviewFactory $orderPreviewFactory + * @param \Shopsys\FrameworkBundle\Component\FlashMessage\ErrorService $errorService + */ public function __construct( ProductAccessoryFacade $productAccessoryFacade, CartFacade $cartFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Front/CategoryController.php b/src/Shopsys/ShopBundle/Controller/Front/CategoryController.php index 1aa0e048d..4832468d1 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/CategoryController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/CategoryController.php @@ -36,6 +36,13 @@ class CategoryController extends FrontBaseController */ private $currentCustomer; + /** + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Category\CategoryFacade $categoryFacade + * @param \Shopsys\ShopBundle\Model\Category\CurrentCategoryResolver $currentCategoryResolver + * @param \Shopsys\FrameworkBundle\Model\Category\TopCategory\TopCategoryFacade $topCategoryFacade + * @param \Shopsys\FrameworkBundle\Model\Customer\CurrentCustomer $currentCustomer + */ public function __construct( Domain $domain, CategoryFacade $categoryFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Front/ContactFormController.php b/src/Shopsys/ShopBundle/Controller/Front/ContactFormController.php index 83c9658b7..84cbca17d 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/ContactFormController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/ContactFormController.php @@ -27,6 +27,11 @@ class ContactFormController extends FrontBaseController */ private $domain; + /** + * @param \Shopsys\FrameworkBundle\Model\ContactForm\ContactFormFacade $contactFormFacade + * @param \Shopsys\FrameworkBundle\Model\LegalConditions\LegalConditionsFacade $legalConditionsFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + */ public function __construct( ContactFormFacade $contactFormFacade, LegalConditionsFacade $legalConditionsFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Front/CustomerController.php b/src/Shopsys/ShopBundle/Controller/Front/CustomerController.php index cb24be973..d6540a005 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/CustomerController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/CustomerController.php @@ -44,6 +44,14 @@ class CustomerController extends FrontBaseController */ private $customerDataFactory; + /** + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerFacade $customerFacade + * @param \Shopsys\FrameworkBundle\Model\Order\OrderFacade $orderFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Order\Item\OrderItemPriceCalculation $orderItemPriceCalculation + * @param \Shopsys\FrameworkBundle\Model\Security\LoginAsUserFacade $loginAsUserFacade + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerDataFactoryInterface $customerDataFactory + */ public function __construct( CustomerFacade $customerFacade, OrderFacade $orderFacade, @@ -60,6 +68,9 @@ public function __construct( $this->customerDataFactory = $customerDataFactory; } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function editAction(Request $request) { if (!$this->isGranted(Roles::ROLE_LOGGED_CUSTOMER)) { diff --git a/src/Shopsys/ShopBundle/Controller/Front/CustomerPasswordController.php b/src/Shopsys/ShopBundle/Controller/Front/CustomerPasswordController.php index 4a6ae23b7..986373862 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/CustomerPasswordController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/CustomerPasswordController.php @@ -26,6 +26,11 @@ class CustomerPasswordController extends FrontBaseController */ private $loginService; + /** + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerPasswordFacade $customerPasswordFacade + * @param \Shopsys\FrameworkBundle\Model\Security\LoginService $loginService + */ public function __construct( Domain $domain, CustomerPasswordFacade $customerPasswordFacade, @@ -36,6 +41,9 @@ public function __construct( $this->loginService = $loginService; } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function resetPasswordAction(Request $request) { $form = $this->createForm(ResetPasswordFormType::class); @@ -72,6 +80,9 @@ public function resetPasswordAction(Request $request) ]); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function setNewPasswordAction(Request $request) { $email = $request->query->get('email'); diff --git a/src/Shopsys/ShopBundle/Controller/Front/ErrorController.php b/src/Shopsys/ShopBundle/Controller/Front/ErrorController.php index 64d93653b..f71c95484 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/ErrorController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/ErrorController.php @@ -36,6 +36,12 @@ class ErrorController extends FrontBaseController */ private $domain; + /** + * @param \Shopsys\FrameworkBundle\Component\Error\ExceptionController $exceptionController + * @param \Shopsys\FrameworkBundle\Component\Error\ExceptionListener $exceptionListener + * @param \Shopsys\FrameworkBundle\Component\Error\ErrorPagesFacade $errorPagesFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + */ public function __construct( ExceptionController $exceptionController, ExceptionListener $exceptionListener, diff --git a/src/Shopsys/ShopBundle/Controller/Front/HomepageController.php b/src/Shopsys/ShopBundle/Controller/Front/HomepageController.php index bd53bb6d0..06d4898c5 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/HomepageController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/HomepageController.php @@ -35,6 +35,13 @@ class HomepageController extends FrontBaseController */ private $domain; + /** + * @param \Shopsys\FrameworkBundle\Model\Customer\CurrentCustomer $currentCustomer + * @param \Shopsys\FrameworkBundle\Model\Slider\SliderItemFacade $sliderItemFacade + * @param \Shopsys\FrameworkBundle\Model\Product\TopProduct\TopProductFacade $topProductsFacade + * @param \Shopsys\FrameworkBundle\Model\Seo\SeoSettingFacade $seoSettingFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + */ public function __construct( CurrentCustomer $currentCustomer, SliderItemFacade $sliderItemFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Front/ImageController.php b/src/Shopsys/ShopBundle/Controller/Front/ImageController.php index 77cf67d05..ae0b3f715 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/ImageController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/ImageController.php @@ -19,12 +19,22 @@ class ImageController extends FrontBaseController */ private $filesystem; + /** + * @param \Shopsys\FrameworkBundle\Component\Image\Processing\ImageGeneratorFacade $imageGeneratorFacade + * @param \League\Flysystem\FilesystemInterface $filesystem + */ public function __construct(ImageGeneratorFacade $imageGeneratorFacade, FilesystemInterface $filesystem) { $this->imageGeneratorFacade = $imageGeneratorFacade; $this->filesystem = $filesystem; } + /** + * @param mixed $entityName + * @param mixed $type + * @param mixed $sizeName + * @param mixed $imageId + */ public function getImageAction($entityName, $type, $sizeName, $imageId) { if ($sizeName === ImageConfig::DEFAULT_SIZE_NAME) { diff --git a/src/Shopsys/ShopBundle/Controller/Front/LoginController.php b/src/Shopsys/ShopBundle/Controller/Front/LoginController.php index 1b58fb0cc..a45007c41 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/LoginController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/LoginController.php @@ -15,11 +15,17 @@ class LoginController extends FrontBaseController */ private $loginService; + /** + * @param \Shopsys\FrameworkBundle\Model\Security\LoginService $loginService + */ public function __construct(LoginService $loginService) { $this->loginService = $loginService; } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function loginAction(Request $request) { if ($this->isGranted(Roles::ROLE_LOGGED_CUSTOMER)) { diff --git a/src/Shopsys/ShopBundle/Controller/Front/NewsletterController.php b/src/Shopsys/ShopBundle/Controller/Front/NewsletterController.php index d2e308146..52b5f8118 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/NewsletterController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/NewsletterController.php @@ -27,6 +27,11 @@ class NewsletterController extends FrontBaseController */ private $domain; + /** + * @param \Shopsys\FrameworkBundle\Model\Newsletter\NewsletterFacade $newsletterFacade + * @param \Shopsys\FrameworkBundle\Model\LegalConditions\LegalConditionsFacade $legalConditionsFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + */ public function __construct( NewsletterFacade $newsletterFacade, LegalConditionsFacade $legalConditionsFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Front/OrderController.php b/src/Shopsys/ShopBundle/Controller/Front/OrderController.php index fbb8cd654..e71bd9ae3 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/OrderController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/OrderController.php @@ -111,6 +111,24 @@ class OrderController extends FrontBaseController */ private $newsletterFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Order\OrderFacade $orderFacade + * @param \Shopsys\FrameworkBundle\Model\Cart\CartFacade $cartFacade + * @param \Shopsys\FrameworkBundle\Model\Order\Preview\OrderPreviewFactory $orderPreviewFactory + * @param \Shopsys\FrameworkBundle\Model\Transport\TransportPriceCalculation $transportPriceCalculation + * @param \Shopsys\FrameworkBundle\Model\Payment\PaymentPriceCalculation $paymentPriceCalculation + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Transport\TransportFacade $transportFacade + * @param \Shopsys\FrameworkBundle\Model\Payment\PaymentFacade $paymentFacade + * @param \Shopsys\FrameworkBundle\Model\Pricing\Currency\CurrencyFacade $currencyFacade + * @param \Shopsys\FrameworkBundle\Model\Order\OrderDataMapper $orderDataMapper + * @param \Shopsys\ShopBundle\Form\Front\Order\DomainAwareOrderFlowFactory $domainAwareOrderFlowFactory + * @param \Symfony\Component\HttpFoundation\Session\SessionInterface $session + * @param \Shopsys\FrameworkBundle\Model\Order\Watcher\TransportAndPaymentWatcherService $transportAndPaymentWatcherService + * @param \Shopsys\FrameworkBundle\Model\Order\Mail\OrderMailFacade $orderMailFacade + * @param \Shopsys\FrameworkBundle\Model\LegalConditions\LegalConditionsFacade $legalConditionsFacade + * @param \Shopsys\FrameworkBundle\Model\Newsletter\NewsletterFacade $newsletterFacade + */ public function __construct( OrderFacade $orderFacade, CartFacade $cartFacade, diff --git a/src/Shopsys/ShopBundle/Controller/Front/PersonalDataController.php b/src/Shopsys/ShopBundle/Controller/Front/PersonalDataController.php index 64e6808dd..d5d2b674b 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/PersonalDataController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/PersonalDataController.php @@ -63,6 +63,17 @@ class PersonalDataController extends FrontBaseController */ private $xmlResponse; + /** + * @param \Shopsys\FrameworkBundle\Component\Setting\Setting $setting + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerFacade $customerFacade + * @param \Shopsys\FrameworkBundle\Model\Order\OrderFacade $orderFacade + * @param \Shopsys\FrameworkBundle\Model\Newsletter\NewsletterFacade $newsletterFacade + * @param \Shopsys\FrameworkBundle\Model\PersonalData\Mail\PersonalDataAccessMailFacade $personalDataAccessMailFacade + * @param \Shopsys\FrameworkBundle\Model\PersonalData\PersonalDataAccessRequestFacade $personalDataAccessRequestFacade + * @param \Shopsys\FrameworkBundle\Model\PersonalData\PersonalDataAccessRequestDataFactoryInterface $personalDataAccessRequestDataFactory + * @param \Shopsys\FrameworkBundle\Component\HttpFoundation\XmlResponse $xmlResponse + */ public function __construct( Setting $setting, Domain $domain, @@ -85,6 +96,9 @@ public function __construct( $this->xmlResponse = $xmlResponse; } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function indexAction(Request $request) { $form = $this->createForm( @@ -112,6 +126,9 @@ public function indexAction(Request $request) ]); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function exportAction(Request $request) { $form = $this->createForm( diff --git a/src/Shopsys/ShopBundle/Controller/Front/ProductController.php b/src/Shopsys/ShopBundle/Controller/Front/ProductController.php index f7f5e1b1a..a569d4a73 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/ProductController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/ProductController.php @@ -75,6 +75,18 @@ class ProductController extends FrontBaseController */ private $brandFacade; + /** + * @param \Shopsys\FrameworkBundle\Twig\RequestExtension $requestExtension + * @param \Shopsys\FrameworkBundle\Model\Category\CategoryFacade $categoryFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Product\ProductOnCurrentDomainFacade $productOnCurrentDomainFacade + * @param \Shopsys\FrameworkBundle\Model\Product\Filter\ProductFilterConfigFactory $productFilterConfigFactory + * @param \Shopsys\FrameworkBundle\Model\Product\Listing\ProductListOrderingModeForListFacade $productListOrderingModeForListFacade + * @param \Shopsys\FrameworkBundle\Model\Product\Listing\ProductListOrderingModeForBrandFacade $productListOrderingModeForBrandFacade + * @param \Shopsys\FrameworkBundle\Model\Product\Listing\ProductListOrderingModeForSearchFacade $productListOrderingModeForSearchFacade + * @param \Shopsys\FrameworkBundle\Model\Module\ModuleFacade $moduleFacade + * @param \Shopsys\FrameworkBundle\Model\Product\Brand\BrandFacade $brandFacade + */ public function __construct( RequestExtension $requestExtension, CategoryFacade $categoryFacade, @@ -320,6 +332,9 @@ private function searchCategories($searchText) ); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function selectOrderingModeForListAction(Request $request) { $productListOrderingConfig = $this->productListOrderingModeForListFacade->getProductListOrderingConfig(); @@ -335,6 +350,9 @@ public function selectOrderingModeForListAction(Request $request) ]); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function selectOrderingModeForListByBrandAction(Request $request) { $productListOrderingConfig = $this->productListOrderingModeForBrandFacade->getProductListOrderingConfig(); @@ -350,6 +368,9 @@ public function selectOrderingModeForListByBrandAction(Request $request) ]); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function selectOrderingModeForSearchAction(Request $request) { $productListOrderingConfig = $this->productListOrderingModeForSearchFacade->getProductListOrderingConfig(); diff --git a/src/Shopsys/ShopBundle/Controller/Front/PromoCodeController.php b/src/Shopsys/ShopBundle/Controller/Front/PromoCodeController.php index ec51db452..0749c9a32 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/PromoCodeController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/PromoCodeController.php @@ -15,6 +15,9 @@ class PromoCodeController extends FrontBaseController */ private $currentPromoCodeFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Order\PromoCode\CurrentPromoCodeFacade $currentPromoCodeFacade + */ public function __construct( CurrentPromoCodeFacade $currentPromoCodeFacade ) { diff --git a/src/Shopsys/ShopBundle/Controller/Front/RegistrationController.php b/src/Shopsys/ShopBundle/Controller/Front/RegistrationController.php index e84a13208..1595309c9 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/RegistrationController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/RegistrationController.php @@ -38,6 +38,13 @@ class RegistrationController extends FrontBaseController */ private $legalConditionsFacade; + /** + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Customer\UserDataFactoryInterface $userDataFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerFacade $customerFacade + * @param \Shopsys\FrameworkBundle\Model\Security\LoginService $loginService + * @param \Shopsys\FrameworkBundle\Model\LegalConditions\LegalConditionsFacade $legalConditionsFacade + */ public function __construct( Domain $domain, UserDataFactoryInterface $userDataFactory, @@ -52,6 +59,9 @@ public function __construct( $this->legalConditionsFacade = $legalConditionsFacade; } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function existsEmailAction(Request $request) { $email = $request->get('email'); @@ -60,6 +70,9 @@ public function existsEmailAction(Request $request) return new JsonResponse($user !== null); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function registerAction(Request $request) { $userData = $this->userDataFactory->createForDomainId($this->domain->getId()); diff --git a/src/Shopsys/ShopBundle/Controller/Front/RobotsController.php b/src/Shopsys/ShopBundle/Controller/Front/RobotsController.php index 2be050882..132f65138 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/RobotsController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/RobotsController.php @@ -18,6 +18,10 @@ class RobotsController extends FrontBaseController */ private $sitemapService; + /** + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Sitemap\SitemapService $sitemapService + */ public function __construct( Domain $domain, SitemapService $sitemapService diff --git a/src/Shopsys/ShopBundle/Controller/Front/ScriptController.php b/src/Shopsys/ShopBundle/Controller/Front/ScriptController.php index f749a5dc9..9739c9f14 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/ScriptController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/ScriptController.php @@ -19,6 +19,10 @@ class ScriptController extends FrontBaseController */ private $domain; + /** + * @param \Shopsys\FrameworkBundle\Model\Script\ScriptFacade $scriptFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + */ public function __construct( ScriptFacade $scriptFacade, Domain $domain diff --git a/src/Shopsys/ShopBundle/Controller/Front/SearchController.php b/src/Shopsys/ShopBundle/Controller/Front/SearchController.php index 520e84e72..7dbe543bc 100644 --- a/src/Shopsys/ShopBundle/Controller/Front/SearchController.php +++ b/src/Shopsys/ShopBundle/Controller/Front/SearchController.php @@ -21,6 +21,10 @@ class SearchController extends FrontBaseController */ private $productOnCurrentDomainFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Category\CategoryFacade $categoryFacade + * @param \Shopsys\FrameworkBundle\Model\Product\ProductOnCurrentDomainFacade $productOnCurrentDomainFacade + */ public function __construct( CategoryFacade $categoryFacade, ProductOnCurrentDomainFacade $productOnCurrentDomainFacade @@ -29,6 +33,9 @@ public function __construct( $this->productOnCurrentDomainFacade = $productOnCurrentDomainFacade; } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function autocompleteAction(Request $request) { $searchText = $request->get('searchText'); @@ -47,6 +54,9 @@ public function autocompleteAction(Request $request) ]); } + /** + * @param \Symfony\Component\HttpFoundation\Request $request + */ public function boxAction(Request $request) { $searchText = $request->query->get(ProductController::SEARCH_TEXT_PARAMETER); diff --git a/src/Shopsys/ShopBundle/DataFixtures/Demo/MailTemplateDataFixture.php b/src/Shopsys/ShopBundle/DataFixtures/Demo/MailTemplateDataFixture.php index 1db1fb69c..116ff2d2a 100644 --- a/src/Shopsys/ShopBundle/DataFixtures/Demo/MailTemplateDataFixture.php +++ b/src/Shopsys/ShopBundle/DataFixtures/Demo/MailTemplateDataFixture.php @@ -27,6 +27,11 @@ class MailTemplateDataFixture extends AbstractReferenceFixture */ protected $mailTemplateDataFactory; + /** + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Mail\MailTemplateFactoryInterface $mailTemplateFactory + * @param \Shopsys\FrameworkBundle\Model\Mail\MailTemplateDataFactoryInterface $mailTemplateDataFactory + */ public function __construct( Domain $domain, MailTemplateFactoryInterface $mailTemplateFactory, @@ -354,6 +359,11 @@ public function load(ObjectManager $manager) $this->createMailTemplate($manager, MailTemplate::PERSONAL_DATA_EXPORT_NAME, $mailTemplateData); } + /** + * @param \Doctrine\Common\Persistence\ObjectManager $manager + * @param mixed $name + * @param \Shopsys\FrameworkBundle\Model\Mail\MailTemplateData $mailTemplateData + */ private function createMailTemplate( ObjectManager $manager, $name, diff --git a/src/Shopsys/ShopBundle/DataFixtures/DemoMultidomain/MailTemplateDataFixture.php b/src/Shopsys/ShopBundle/DataFixtures/DemoMultidomain/MailTemplateDataFixture.php index a2a799de4..4a2663089 100644 --- a/src/Shopsys/ShopBundle/DataFixtures/DemoMultidomain/MailTemplateDataFixture.php +++ b/src/Shopsys/ShopBundle/DataFixtures/DemoMultidomain/MailTemplateDataFixture.php @@ -29,6 +29,11 @@ class MailTemplateDataFixture extends AbstractReferenceFixture */ private $mailTemplateDataFactory; + /** + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Shopsys\FrameworkBundle\Model\Mail\MailTemplateFacade $mailTemplateFacade + * @param \Shopsys\FrameworkBundle\Model\Mail\MailTemplateDataFactoryInterface $mailTemplateDataFactory + */ public function __construct( Domain $domain, MailTemplateFacade $mailTemplateFacade, diff --git a/src/Shopsys/ShopBundle/Form/Admin/CategoryFormTypeExtension.php b/src/Shopsys/ShopBundle/Form/Admin/CategoryFormTypeExtension.php index 50704541d..34102545c 100644 --- a/src/Shopsys/ShopBundle/Form/Admin/CategoryFormTypeExtension.php +++ b/src/Shopsys/ShopBundle/Form/Admin/CategoryFormTypeExtension.php @@ -37,10 +37,10 @@ public function buildForm(FormBuilderInterface $builder, array $options) } /** - * @param FormBuilderInterface $builder + * @param \Symfony\Component\Form\FormBuilderInterface $builder * @param string $name * @param array $options - * @return FormBuilderInterface + * @return \Symfony\Component\Form\FormBuilderInterface */ protected function rebuildChildWithAdditionalOptions(FormBuilderInterface $builder, string $name, array $options): FormBuilderInterface { diff --git a/src/Shopsys/ShopBundle/Form/Admin/ProductFormTypeExtension.php b/src/Shopsys/ShopBundle/Form/Admin/ProductFormTypeExtension.php index 09fb89981..303a7ce99 100644 --- a/src/Shopsys/ShopBundle/Form/Admin/ProductFormTypeExtension.php +++ b/src/Shopsys/ShopBundle/Form/Admin/ProductFormTypeExtension.php @@ -15,6 +15,9 @@ class ProductFormTypeExtension extends AbstractTypeExtension */ private $productConditionFacade; + /** + * @param \Shopsys\ShopBundle\Model\Product\ProductConditionFacade $productConditionFacade + */ public function __construct(ProductConditionFacade $productConditionFacade) { $this->productConditionFacade = $productConditionFacade; diff --git a/src/Shopsys/ShopBundle/Form/Front/Customer/BillingAddressFormType.php b/src/Shopsys/ShopBundle/Form/Front/Customer/BillingAddressFormType.php index 0d94ed726..a5eaced31 100644 --- a/src/Shopsys/ShopBundle/Form/Front/Customer/BillingAddressFormType.php +++ b/src/Shopsys/ShopBundle/Form/Front/Customer/BillingAddressFormType.php @@ -23,6 +23,9 @@ class BillingAddressFormType extends AbstractType */ private $countryFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Country\CountryFacade $countryFacade + */ public function __construct(CountryFacade $countryFacade) { $this->countryFacade = $countryFacade; diff --git a/src/Shopsys/ShopBundle/Form/Front/Customer/CustomerFormType.php b/src/Shopsys/ShopBundle/Form/Front/Customer/CustomerFormType.php index 41e670f39..8eefbfa1a 100644 --- a/src/Shopsys/ShopBundle/Form/Front/Customer/CustomerFormType.php +++ b/src/Shopsys/ShopBundle/Form/Front/Customer/CustomerFormType.php @@ -15,6 +15,9 @@ class CustomerFormType extends AbstractType */ private $customerDataFactory; + /** + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerDataFactoryInterface $customerDataFactory + */ public function __construct(CustomerDataFactoryInterface $customerDataFactory) { $this->customerDataFactory = $customerDataFactory; diff --git a/src/Shopsys/ShopBundle/Form/Front/Customer/DeliveryAddressFormType.php b/src/Shopsys/ShopBundle/Form/Front/Customer/DeliveryAddressFormType.php index bf4a45def..f275974f2 100644 --- a/src/Shopsys/ShopBundle/Form/Front/Customer/DeliveryAddressFormType.php +++ b/src/Shopsys/ShopBundle/Form/Front/Customer/DeliveryAddressFormType.php @@ -23,6 +23,9 @@ class DeliveryAddressFormType extends AbstractType */ private $countryFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Country\CountryFacade $countryFacade + */ public function __construct(CountryFacade $countryFacade) { $this->countryFacade = $countryFacade; diff --git a/src/Shopsys/ShopBundle/Form/Front/Order/DomainAwareOrderFlowFactory.php b/src/Shopsys/ShopBundle/Form/Front/Order/DomainAwareOrderFlowFactory.php index c390d5c18..0d958476b 100644 --- a/src/Shopsys/ShopBundle/Form/Front/Order/DomainAwareOrderFlowFactory.php +++ b/src/Shopsys/ShopBundle/Form/Front/Order/DomainAwareOrderFlowFactory.php @@ -38,6 +38,10 @@ class DomainAwareOrderFlowFactory implements OrderFlowFactoryInterface /** * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + * @param \Symfony\Component\EventDispatcher\EventDispatcherInterface $eventDispatcher + * @param \Symfony\Component\HttpFoundation\RequestStack $requestStack + * @param \Symfony\Component\Form\FormFactoryInterface $formFactory + * @param \Craue\FormFlowBundle\Storage\DataManager $dataManager */ public function __construct( Domain $domain, diff --git a/src/Shopsys/ShopBundle/Form/Front/Order/OrderFlow.php b/src/Shopsys/ShopBundle/Form/Front/Order/OrderFlow.php index 333b52179..8b19f7364 100644 --- a/src/Shopsys/ShopBundle/Form/Front/Order/OrderFlow.php +++ b/src/Shopsys/ShopBundle/Form/Front/Order/OrderFlow.php @@ -119,7 +119,7 @@ public function bind($formData) } /** - * @return StepInterface|null + * @return \Craue\FormFlowBundle\Form\StepInterface|null */ private function getFirstInvalidStep() { diff --git a/src/Shopsys/ShopBundle/Form/Front/Order/PersonalInfoFormType.php b/src/Shopsys/ShopBundle/Form/Front/Order/PersonalInfoFormType.php index 3436266e9..6388bfc56 100644 --- a/src/Shopsys/ShopBundle/Form/Front/Order/PersonalInfoFormType.php +++ b/src/Shopsys/ShopBundle/Form/Front/Order/PersonalInfoFormType.php @@ -42,6 +42,11 @@ class PersonalInfoFormType extends AbstractType */ private $domain; + /** + * @param \Shopsys\FrameworkBundle\Model\Country\CountryFacade $countryFacade + * @param \Shopsys\FrameworkBundle\Model\Heureka\HeurekaFacade $heurekaFacade + * @param \Shopsys\FrameworkBundle\Component\Domain\Domain $domain + */ public function __construct(CountryFacade $countryFacade, HeurekaFacade $heurekaFacade, Domain $domain) { $this->countryFacade = $countryFacade; diff --git a/src/Shopsys/ShopBundle/Model/AdminNavigation/SideMenuBuilder.php b/src/Shopsys/ShopBundle/Model/AdminNavigation/SideMenuBuilder.php index cbe204101..10c036052 100644 --- a/src/Shopsys/ShopBundle/Model/AdminNavigation/SideMenuBuilder.php +++ b/src/Shopsys/ShopBundle/Model/AdminNavigation/SideMenuBuilder.php @@ -7,6 +7,9 @@ class SideMenuBuilder extends BaseSideMenuBuilder { + /** + * @return \Knp\Menu\ItemInterface + */ protected function createSettingsMenu(): ItemInterface { $menu = parent::createSettingsMenu(); diff --git a/src/Shopsys/ShopBundle/Model/Category/CurrentCategoryResolver.php b/src/Shopsys/ShopBundle/Model/Category/CurrentCategoryResolver.php index 5f22d3c6a..992eb2a77 100644 --- a/src/Shopsys/ShopBundle/Model/Category/CurrentCategoryResolver.php +++ b/src/Shopsys/ShopBundle/Model/Category/CurrentCategoryResolver.php @@ -18,6 +18,10 @@ class CurrentCategoryResolver */ private $productFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Category\CategoryFacade $categoryFacade + * @param \Shopsys\FrameworkBundle\Model\Product\ProductFacade $productFacade + */ public function __construct( CategoryFacade $categoryFacade, ProductFacade $productFacade diff --git a/src/Shopsys/ShopBundle/Model/Customer/BillingAddressFacade.php b/src/Shopsys/ShopBundle/Model/Customer/BillingAddressFacade.php index 584303aab..c284e593c 100644 --- a/src/Shopsys/ShopBundle/Model/Customer/BillingAddressFacade.php +++ b/src/Shopsys/ShopBundle/Model/Customer/BillingAddressFacade.php @@ -17,6 +17,7 @@ class BillingAddressFacade protected $billingAddressRepository; /** + * @param \Doctrine\ORM\EntityManagerInterface $em * @param \Shopsys\ShopBundle\Model\Customer\BillingAddressRepository $billingAddressRepository */ public function __construct( diff --git a/src/Shopsys/ShopBundle/Model/Customer/CustomerData.php b/src/Shopsys/ShopBundle/Model/Customer/CustomerData.php index 9ad621761..d235e1de3 100644 --- a/src/Shopsys/ShopBundle/Model/Customer/CustomerData.php +++ b/src/Shopsys/ShopBundle/Model/Customer/CustomerData.php @@ -14,6 +14,11 @@ class CustomerData extends BaseCustomerData */ public $companyUsersData; + /** + * @param \Shopsys\FrameworkBundle\Model\Customer\BillingAddressData $billingAddressData + * @param \Shopsys\FrameworkBundle\Model\Customer\DeliveryAddressData $deliveryAddressData + * @param \Shopsys\FrameworkBundle\Model\Customer\UserData $userData + */ public function __construct( BillingAddressData $billingAddressData, DeliveryAddressData $deliveryAddressData, diff --git a/src/Shopsys/ShopBundle/Model/Customer/CustomerFacade.php b/src/Shopsys/ShopBundle/Model/Customer/CustomerFacade.php index 2f9825fd1..3f6688988 100644 --- a/src/Shopsys/ShopBundle/Model/Customer/CustomerFacade.php +++ b/src/Shopsys/ShopBundle/Model/Customer/CustomerFacade.php @@ -32,6 +32,8 @@ class CustomerFacade extends BaseCustomerFacade * @param \Shopsys\FrameworkBundle\Model\Customer\Mail\CustomerMailFacade $customerMailFacade * @param \Shopsys\FrameworkBundle\Model\Customer\BillingAddressFactoryInterface $billingAddressFactory * @param \Shopsys\FrameworkBundle\Model\Customer\BillingAddressDataFactoryInterface $billingAddressDataFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\CustomerDataFactoryInterface $customerDataFactory + * @param \Shopsys\FrameworkBundle\Model\Customer\UserFactoryInterface $userFactory */ public function __construct( EntityManagerInterface $em, diff --git a/src/Shopsys/ShopBundle/Model/Customer/Exception/DuplicateEmailsException.php b/src/Shopsys/ShopBundle/Model/Customer/Exception/DuplicateEmailsException.php index 3ba107a36..6fabe3014 100644 --- a/src/Shopsys/ShopBundle/Model/Customer/Exception/DuplicateEmailsException.php +++ b/src/Shopsys/ShopBundle/Model/Customer/Exception/DuplicateEmailsException.php @@ -13,6 +13,7 @@ class DuplicateEmailsException extends Exception implements CustomerException private $email; /** + * @param mixed $email * @param string $message * @param \Exception|null $previous */ diff --git a/src/Shopsys/ShopBundle/Model/Order/OrderDataMapper.php b/src/Shopsys/ShopBundle/Model/Order/OrderDataMapper.php index 671f58957..2a1092791 100644 --- a/src/Shopsys/ShopBundle/Model/Order/OrderDataMapper.php +++ b/src/Shopsys/ShopBundle/Model/Order/OrderDataMapper.php @@ -15,6 +15,10 @@ class OrderDataMapper extends BaseOrderDataMapper */ protected $countryFacade; + /** + * @param \Shopsys\FrameworkBundle\Model\Order\OrderDataFactoryInterface $orderDataFactory + * @param \Shopsys\ShopBundle\Model\Country\CountryFacade $countryFacade + */ public function __construct(OrderDataFactoryInterface $orderDataFactory, CountryFacade $countryFacade) { parent::__construct($orderDataFactory); @@ -41,6 +45,10 @@ public function getOrderDataFromFrontOrderData(BaseFrontOrderData $frontOrderDat return $orderData; } + /** + * @param \Shopsys\ShopBundle\Model\Order\OrderData $orderData + * @param \Shopsys\ShopBundle\Model\Order\FrontOrderData $frontOrderData + */ private function fillPickUpPlaceDeliveryData(OrderData $orderData, FrontOrderData $frontOrderData) { $orderData->deliveryFirstName = $frontOrderData->firstName; diff --git a/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceFacade.php b/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceFacade.php index 8b743b254..2d48d09db 100644 --- a/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceFacade.php +++ b/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceFacade.php @@ -29,6 +29,12 @@ class PickUpPlaceFacade */ protected $countryFacade; + /** + * @param \Shopsys\ShopBundle\Model\PickUpPlace\PickUpPlaceRepository $pickUpPlaceRepository + * @param \Doctrine\ORM\EntityManagerInterface $em + * @param \Shopsys\ShopBundle\Model\PickUpPlace\PickUpPlaceLoader $pickUpPlaceLoader + * @param \Shopsys\ShopBundle\Model\Country\CountryFacade $countryFacade + */ public function __construct( PickUpPlaceRepository $pickUpPlaceRepository, EntityManagerInterface $em, diff --git a/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceIdToEntityTransformer.php b/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceIdToEntityTransformer.php index 6fd33adaa..8bd49a0e9 100644 --- a/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceIdToEntityTransformer.php +++ b/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceIdToEntityTransformer.php @@ -12,6 +12,9 @@ class PickUpPlaceIdToEntityTransformer implements DataTransformerInterface */ protected $pickUpPlaceFacade; + /** + * @param \Shopsys\ShopBundle\Model\PickUpPlace\PickUpPlaceFacade $pickUpPlaceFacade + */ public function __construct(PickUpPlaceFacade $pickUpPlaceFacade) { $this->pickUpPlaceFacade = $pickUpPlaceFacade; @@ -19,6 +22,7 @@ public function __construct(PickUpPlaceFacade $pickUpPlaceFacade) /** * @var \Shopsys\ShopBundle\Model\PickUpPlace\PickUpPlace + * @param mixed $pickUpPlace * @return int|null */ public function transform($pickUpPlace) @@ -31,6 +35,7 @@ public function transform($pickUpPlace) /** * @var int|null + * @param mixed $pickUpPlaceId * @return null|\Shopsys\ShopBundle\Model\PickUpPlace\PickUpPlace */ public function reverseTransform($pickUpPlaceId) diff --git a/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceRepository.php b/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceRepository.php index 933cc7ce3..e26269d3a 100644 --- a/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceRepository.php +++ b/src/Shopsys/ShopBundle/Model/PickUpPlace/PickUpPlaceRepository.php @@ -13,6 +13,9 @@ class PickUpPlaceRepository */ protected $em; + /** + * @param \Doctrine\ORM\EntityManagerInterface $em + */ public function __construct( EntityManagerInterface $em ) { @@ -118,6 +121,7 @@ public function deactivateAllPendingByTransportType($transportType) /** * @param string|null $searchQuery + * @param mixed $countryCodes * @param $transportType * @return \Shopsys\ShopBundle\Model\PickUpPlace\PickUpPlace[] */ diff --git a/src/Shopsys/ShopBundle/Model/Product/Feed/Google/GoogleFeedItem.php b/src/Shopsys/ShopBundle/Model/Product/Feed/Google/GoogleFeedItem.php index 4c42f372c..da6cca5c0 100644 --- a/src/Shopsys/ShopBundle/Model/Product/Feed/Google/GoogleFeedItem.php +++ b/src/Shopsys/ShopBundle/Model/Product/Feed/Google/GoogleFeedItem.php @@ -13,6 +13,20 @@ class GoogleFeedItem extends BaseGoogleFeedItem */ private $condition; + /** + * @param int $id + * @param string $name + * @param string|null $brandName + * @param string|null $description + * @param string|null $ean + * @param string|null $partno + * @param string $url + * @param string|null $imgUrl + * @param bool $sellingDenied + * @param \Shopsys\FrameworkBundle\Model\Pricing\Price $price + * @param \Shopsys\FrameworkBundle\Model\Pricing\Currency\Currency $currency + * @param string $condition + */ public function __construct( int $id, string $name, diff --git a/src/Shopsys/ShopBundle/Model/Product/Product.php b/src/Shopsys/ShopBundle/Model/Product/Product.php index 0ab7e12cc..6dcbf50b8 100644 --- a/src/Shopsys/ShopBundle/Model/Product/Product.php +++ b/src/Shopsys/ShopBundle/Model/Product/Product.php @@ -51,6 +51,7 @@ public static function createMainVariant(BaseProductData $productData, array $va /** * @param \Shopsys\FrameworkBundle\Model\Product\ProductCategoryDomainFactoryInterface $productCategoryDomainFactory + * @param \Shopsys\FrameworkBundle\Model\Product\ProductData $productData * @param \Shopsys\ShopBundle\Model\Product\ProductData */ public function edit( diff --git a/tests/ShopBundle/Acceptance/acceptance/AdminProductAdvancedSearchCest.php b/tests/ShopBundle/Acceptance/acceptance/AdminProductAdvancedSearchCest.php index ecaab2580..7320850b4 100644 --- a/tests/ShopBundle/Acceptance/acceptance/AdminProductAdvancedSearchCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/AdminProductAdvancedSearchCest.php @@ -8,6 +8,11 @@ class AdminProductAdvancedSearchCest { + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\ProductAdvancedSearchPage $productAdvancedSearchPage + */ public function testSearchByCatnum( AcceptanceTester $me, LoginPage $loginPage, diff --git a/tests/ShopBundle/Acceptance/acceptance/AdministratorLoginCest.php b/tests/ShopBundle/Acceptance/acceptance/AdministratorLoginCest.php index 22ecd6c46..17933d02f 100644 --- a/tests/ShopBundle/Acceptance/acceptance/AdministratorLoginCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/AdministratorLoginCest.php @@ -7,12 +7,20 @@ class AdministratorLoginCest { + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + */ public function testSuccessfulLogin(AcceptanceTester $me, LoginPage $loginPage) { $me->wantTo('login on admin with valid data'); $loginPage->loginAsAdmin(); } + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + */ public function testLoginWithInvalidUsername(AcceptanceTester $me, LoginPage $loginPage) { $me->wantTo('login on admin with nonexistent username'); @@ -20,6 +28,10 @@ public function testLoginWithInvalidUsername(AcceptanceTester $me, LoginPage $lo $loginPage->assertLoginFailed(); } + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + */ public function testLoginWithInvalidPassword(AcceptanceTester $me, LoginPage $loginPage) { $me->wantTo('login on admin with invalid password'); diff --git a/tests/ShopBundle/Acceptance/acceptance/CartCest.php b/tests/ShopBundle/Acceptance/acceptance/CartCest.php index 51f8c8c0a..588c8b883 100644 --- a/tests/ShopBundle/Acceptance/acceptance/CartCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/CartCest.php @@ -12,6 +12,13 @@ class CartCest { + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductDetailPage $productDetailPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartBoxPage $cartBoxPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\FloatingWindowPage $floatingWindowPage + */ public function testAddingSameProductToCartMakesSum( CartPage $cartPage, ProductDetailPage $productDetailPage, @@ -35,6 +42,13 @@ public function testAddingSameProductToCartMakesSum( $cartPage->assertProductQuantity('22" Sencor SLE 22F46DM4 HELLO KITTY', 6); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartBoxPage $cartBoxPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\FloatingWindowPage $floatingWindowPage + */ public function testAddToCartFromProductListPage( CartPage $cartPage, ProductListPage $productListPage, @@ -52,6 +66,13 @@ public function testAddToCartFromProductListPage( $cartPage->assertProductPrice('Defender 2.0 SPK-480', 'CZK119.00'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\HomepagePage $homepagePage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartBoxPage $cartBoxPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\FloatingWindowPage $floatingWindowPage + */ public function testAddToCartFromHomepage( CartPage $cartPage, HomepagePage $homepagePage, @@ -69,6 +90,12 @@ public function testAddToCartFromHomepage( $cartPage->assertProductPrice('22" Sencor SLE 22F46DM4 HELLO KITTY', 'CZK3,499.00'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductDetailPage $productDetailPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartBoxPage $cartBoxPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\FloatingWindowPage $floatingWindowPage + */ public function testAddToCartFromProductDetail( ProductDetailPage $productDetailPage, CartBoxPage $cartBoxPage, @@ -86,6 +113,11 @@ public function testAddToCartFromProductDetail( $me->see('22" Sencor SLE 22F46DM4 HELLO KITTY'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductDetailPage $productDetailPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testChangeCartItemAndRecalculatePrice( CartPage $cartPage, ProductDetailPage $productDetailPage, @@ -101,6 +133,11 @@ public function testChangeCartItemAndRecalculatePrice( $cartPage->assertTotalPriceWithVat('CZK34,990.00'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductDetailPage $productDetailPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testRemovingItemsFromCart( CartPage $cartPage, ProductDetailPage $productDetailPage, @@ -124,6 +161,13 @@ public function testRemovingItemsFromCart( $me->see('Your cart is unfortunately empty.'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartBoxPage $cartBoxPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductDetailPage $productDetailPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\FloatingWindowPage $floatingWindowPage + */ public function testAddingDistinctProductsToCart( CartPage $cartPage, CartBoxPage $cartBoxPage, @@ -148,6 +192,11 @@ public function testAddingDistinctProductsToCart( $cartPage->assertProductIsInCartByName('Canon PIXMA iP7250'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\CartPage $cartPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductDetailPage $productDetailPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testPricingInCart( CartPage $cartPage, ProductDetailPage $productDetailPage, diff --git a/tests/ShopBundle/Acceptance/acceptance/CustomerLoginCest.php b/tests/ShopBundle/Acceptance/acceptance/CustomerLoginCest.php index 8f3cf4cc3..90a8e2014 100644 --- a/tests/ShopBundle/Acceptance/acceptance/CustomerLoginCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/CustomerLoginCest.php @@ -8,6 +8,11 @@ class CustomerLoginCest { + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LoginPage $loginPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LayoutPage $layoutPage + */ public function testLoginAsCustomerFromMainPage( LoginPage $loginPage, AcceptanceTester $me, @@ -23,6 +28,11 @@ public function testLoginAsCustomerFromMainPage( $me->seeCurrentPageEquals('/'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LoginPage $loginPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LayoutPage $layoutPage + */ public function testLoginAsCustomerFromCategoryPage( LoginPage $loginPage, AcceptanceTester $me, @@ -38,6 +48,11 @@ public function testLoginAsCustomerFromCategoryPage( $me->seeCurrentPageEquals('/'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LoginPage $loginPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LayoutPage $layoutPage + */ public function testLoginAsCustomerFromLoginPage( LoginPage $loginPage, AcceptanceTester $me, diff --git a/tests/ShopBundle/Acceptance/acceptance/CustomerRegistrationCest.php b/tests/ShopBundle/Acceptance/acceptance/CustomerRegistrationCest.php index 1e1c77dca..b5ae4d275 100644 --- a/tests/ShopBundle/Acceptance/acceptance/CustomerRegistrationCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/CustomerRegistrationCest.php @@ -10,6 +10,11 @@ class CustomerRegistrationCest { const MINIMUM_FORM_SUBMIT_WAIT_TIME = 10; + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\RegistrationPage $registrationPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\LayoutPage $layoutPage + */ public function testSuccessfulRegistration( RegistrationPage $registrationPage, AcceptanceTester $me, @@ -25,6 +30,10 @@ public function testSuccessfulRegistration( $me->see('Log out'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\RegistrationPage $registrationPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testAlreadyUsedEmail(RegistrationPage $registrationPage, AcceptanceTester $me) { $me->wantTo('use already used email while registration'); @@ -33,6 +42,10 @@ public function testAlreadyUsedEmail(RegistrationPage $registrationPage, Accepta $registrationPage->seeEmailError('Email no-reply@shopsys.com is already registered'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\RegistrationPage $registrationPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testPasswordMismatch(RegistrationPage $registrationPage, AcceptanceTester $me) { $me->wantTo('use mismatching passwords while registration'); diff --git a/tests/ShopBundle/Acceptance/acceptance/ErrorHandlingCest.php b/tests/ShopBundle/Acceptance/acceptance/ErrorHandlingCest.php index 764c50c31..643fa2968 100644 --- a/tests/ShopBundle/Acceptance/acceptance/ErrorHandlingCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/ErrorHandlingCest.php @@ -6,6 +6,9 @@ class ErrorHandlingCest { + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testDisplayNotice(AcceptanceTester $me) { $me->wantTo('display notice error page'); diff --git a/tests/ShopBundle/Acceptance/acceptance/LoginAsCustomerCest.php b/tests/ShopBundle/Acceptance/acceptance/LoginAsCustomerCest.php index 47404bb5d..d609ea2ec 100644 --- a/tests/ShopBundle/Acceptance/acceptance/LoginAsCustomerCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/LoginAsCustomerCest.php @@ -8,6 +8,11 @@ class LoginAsCustomerCest { + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + * @param \Codeception\Scenario $scenario + */ public function testLoginAsCustomer(AcceptanceTester $me, LoginPage $loginPage, Scenario $scenario) { $me->wantTo('login as a customer from admin'); diff --git a/tests/ShopBundle/Acceptance/acceptance/OrderCest.php b/tests/ShopBundle/Acceptance/acceptance/OrderCest.php index 3f94f57b4..f0cbe288a 100644 --- a/tests/ShopBundle/Acceptance/acceptance/OrderCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/OrderCest.php @@ -11,6 +11,11 @@ class OrderCest { + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\OrderPage $orderPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testFormRemembersPaymentAndTransportWhenClickingBack( ProductListPage $productListPage, OrderPage $orderPage, @@ -35,6 +40,11 @@ public function testFormRemembersPaymentAndTransportWhenClickingBack( $orderPage->assertPaymentIsSelected('Cash on delivery'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\OrderPage $orderPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testFormRemembersPaymentAndTransportWhenGoingDirectlyToUrl( ProductListPage $productListPage, OrderPage $orderPage, @@ -59,6 +69,11 @@ public function testFormRemembersPaymentAndTransportWhenGoingDirectlyToUrl( $orderPage->assertPaymentIsSelected('Cash on delivery'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\OrderPage $orderPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ public function testFormRemembersFirstName(ProductListPage $productListPage, OrderPage $orderPage, AcceptanceTester $me) { $me->wantTo('have my first name remembered by order'); @@ -80,6 +95,12 @@ public function testFormRemembersFirstName(ProductListPage $productListPage, Ord $orderPage->assertFirstNameIsFilled('Jan'); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\OrderPage $orderPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Test\Codeception\Helper\SymfonyHelper $symfonyHelper + */ public function testOrderCanBeCompletedAndHasGoogleAnalyticsTrackingIdInSource( ProductListPage $productListPage, OrderPage $orderPage, @@ -103,6 +124,11 @@ private function setGoogleAnalyticsTrackingId($trackingId, ScriptFacade $scriptF $scriptFacade->setGoogleAnalyticsTrackingId($trackingId, Domain::FIRST_DOMAIN_ID); } + /** + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\OrderPage $orderPage + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + */ private function testOrderCanBeCompleted( ProductListPage $productListPage, OrderPage $orderPage, diff --git a/tests/ShopBundle/Acceptance/acceptance/PageObject/AbstractPage.php b/tests/ShopBundle/Acceptance/acceptance/PageObject/AbstractPage.php index 0afc7e26e..fc191475a 100644 --- a/tests/ShopBundle/Acceptance/acceptance/PageObject/AbstractPage.php +++ b/tests/ShopBundle/Acceptance/acceptance/PageObject/AbstractPage.php @@ -17,6 +17,10 @@ abstract class AbstractPage */ protected $tester; + /** + * @param \Tests\ShopBundle\Test\Codeception\Module\StrictWebDriver $strictWebDriver + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $tester + */ public function __construct(StrictWebDriver $strictWebDriver, AcceptanceTester $tester) { $this->webDriver = $strictWebDriver->webDriver; diff --git a/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/HomepagePage.php b/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/HomepagePage.php index 4ecf9da15..27b31e542 100644 --- a/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/HomepagePage.php +++ b/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/HomepagePage.php @@ -14,6 +14,11 @@ class HomepagePage extends AbstractPage */ private $productListComponent; + /** + * @param \Tests\ShopBundle\Test\Codeception\Module\StrictWebDriver $strictWebDriver + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $tester + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListComponent $productListComponent + */ public function __construct( StrictWebDriver $strictWebDriver, AcceptanceTester $tester, diff --git a/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductFilterPage.php b/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductFilterPage.php index 6a28207e1..4b26756ad 100644 --- a/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductFilterPage.php +++ b/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductFilterPage.php @@ -12,6 +12,10 @@ class ProductFilterPage extends AbstractPage // Product filter waits for more requests before evaluation const PRE_EVALUATION_WAIT = 2; + /** + * @param \Tests\ShopBundle\Test\Codeception\Module\StrictWebDriver $strictWebDriver + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $tester + */ public function __construct(StrictWebDriver $strictWebDriver, AcceptanceTester $tester) { parent::__construct($strictWebDriver, $tester); diff --git a/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductListPage.php b/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductListPage.php index e08ba4ad1..a6afd378e 100644 --- a/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductListPage.php +++ b/tests/ShopBundle/Acceptance/acceptance/PageObject/Front/ProductListPage.php @@ -15,6 +15,11 @@ class ProductListPage extends AbstractPage */ private $productListComponent; + /** + * @param \Tests\ShopBundle\Test\Codeception\Module\StrictWebDriver $strictWebDriver + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $tester + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListComponent $productListComponent + */ public function __construct( StrictWebDriver $strictWebDriver, AcceptanceTester $tester, diff --git a/tests/ShopBundle/Acceptance/acceptance/PaymentImageUploadCest.php b/tests/ShopBundle/Acceptance/acceptance/PaymentImageUploadCest.php index d12084d88..0b69a6a14 100644 --- a/tests/ShopBundle/Acceptance/acceptance/PaymentImageUploadCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/PaymentImageUploadCest.php @@ -15,6 +15,11 @@ class PaymentImageUploadCest const TEST_IMAGE_NAME = 'paymentTestImage.png'; + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\EntityEditPage $entityEditPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + */ public function testSuccessfulImageUpload(AcceptanceTester $me, EntityEditPage $entityEditPage, LoginPage $loginPage) { $me->wantTo('Upload an image in admin payment edit page'); diff --git a/tests/ShopBundle/Acceptance/acceptance/ProductFilterCest.php b/tests/ShopBundle/Acceptance/acceptance/ProductFilterCest.php index 405989b32..93d29c52d 100644 --- a/tests/ShopBundle/Acceptance/acceptance/ProductFilterCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/ProductFilterCest.php @@ -8,6 +8,11 @@ class ProductFilterCest { + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductFilterPage $productFilterPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Front\ProductListPage $productListPage + */ public function testAllProductFilters( AcceptanceTester $me, ProductFilterPage $productFilterPage, diff --git a/tests/ShopBundle/Acceptance/acceptance/ProductImageUploadCest.php b/tests/ShopBundle/Acceptance/acceptance/ProductImageUploadCest.php index 3a461dafe..4ed895039 100644 --- a/tests/ShopBundle/Acceptance/acceptance/ProductImageUploadCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/ProductImageUploadCest.php @@ -15,6 +15,11 @@ class ProductImageUploadCest const TEST_IMAGE_NAME = 'productTestImage.png'; + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\EntityEditPage $entityEditPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + */ public function testSuccessfulImageUpload(AcceptanceTester $me, EntityEditPage $entityEditPage, LoginPage $loginPage) { $me->wantTo('upload image in admin product edit page'); diff --git a/tests/ShopBundle/Acceptance/acceptance/PromoCodeInlineEditCest.php b/tests/ShopBundle/Acceptance/acceptance/PromoCodeInlineEditCest.php index ca80ff72e..9b926fe56 100644 --- a/tests/ShopBundle/Acceptance/acceptance/PromoCodeInlineEditCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/PromoCodeInlineEditCest.php @@ -8,6 +8,11 @@ class PromoCodeInlineEditCest { + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\InlineEditPage $inlineEditPage + */ public function testPromoCodeEdit(AcceptanceTester $me, LoginPage $loginPage, InlineEditPage $inlineEditPage) { $me->wantTo('promo code can be edited via inline edit'); @@ -21,6 +26,11 @@ public function testPromoCodeEdit(AcceptanceTester $me, LoginPage $loginPage, In $inlineEditPage->assertSeeInColumn(1, 'code', 'test edited'); } + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\InlineEditPage $inlineEditPage + */ public function testPromoCodeCreate(AcceptanceTester $me, LoginPage $loginPage, InlineEditPage $inlineEditPage) { $me->wantTo('promo code can be created via inline edit'); @@ -38,6 +48,11 @@ public function testPromoCodeCreate(AcceptanceTester $me, LoginPage $loginPage, $inlineEditPage->assertSeeInColumn($newRowId, 'percent', '5%'); } + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\InlineEditPage $inlineEditPage + */ public function testPromoCodeDelete(AcceptanceTester $me, LoginPage $loginPage, InlineEditPage $inlineEditPage) { $me->wantTo('promo code can be deleted via inline edit'); diff --git a/tests/ShopBundle/Acceptance/acceptance/TransportImageUploadCest.php b/tests/ShopBundle/Acceptance/acceptance/TransportImageUploadCest.php index ab468de68..ab526bd07 100644 --- a/tests/ShopBundle/Acceptance/acceptance/TransportImageUploadCest.php +++ b/tests/ShopBundle/Acceptance/acceptance/TransportImageUploadCest.php @@ -15,6 +15,11 @@ class TransportImageUploadCest const TEST_IMAGE_NAME = 'transportTestImage.png'; + /** + * @param \Tests\ShopBundle\Test\Codeception\AcceptanceTester $me + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\EntityEditPage $entityEditPage + * @param \Tests\ShopBundle\Acceptance\acceptance\PageObject\Admin\LoginPage $loginPage + */ public function testSuccessfulImageUpload(AcceptanceTester $me, EntityEditPage $entityEditPage, LoginPage $loginPage) { $me->wantTo('Upload an image in admin transport edit page'); diff --git a/tests/ShopBundle/Functional/Model/Customer/CustomerPasswordServiceTest.php b/tests/ShopBundle/Functional/Model/Customer/CustomerPasswordServiceTest.php index d66a5523e..7e18d3f88 100644 --- a/tests/ShopBundle/Functional/Model/Customer/CustomerPasswordServiceTest.php +++ b/tests/ShopBundle/Functional/Model/Customer/CustomerPasswordServiceTest.php @@ -48,6 +48,10 @@ public function isResetPasswordHashValidProvider() /** * @dataProvider isResetPasswordHashValidProvider + * @param mixed $resetPasswordHash + * @param mixed $resetPasswordHashValidThrough + * @param mixed $sentHash + * @param mixed $isExpectedValid */ public function testIsResetPasswordHashValid( $resetPasswordHash, diff --git a/tests/ShopBundle/Functional/Model/Order/OrderTransportAndPaymentTest.php b/tests/ShopBundle/Functional/Model/Order/OrderTransportAndPaymentTest.php index b2b1999fd..f3d56175d 100644 --- a/tests/ShopBundle/Functional/Model/Order/OrderTransportAndPaymentTest.php +++ b/tests/ShopBundle/Functional/Model/Order/OrderTransportAndPaymentTest.php @@ -373,7 +373,7 @@ public function testVisiblePaymentTransportOnDifferentDomain() * @param \Shopsys\FrameworkBundle\Model\Pricing\Vat\Vat $vat * @param bool[] $enabledForDomains * @param bool $hidden - * @return Payment + * @return \Shopsys\FrameworkBundle\Model\Payment\Payment */ public function getDefaultPayment(Vat $vat, $enabledForDomains, $hidden) { diff --git a/tests/ShopBundle/Functional/Model/Payment/IndependentPaymentVisibilityCalculationTest.php b/tests/ShopBundle/Functional/Model/Payment/IndependentPaymentVisibilityCalculationTest.php index f67052d1b..dc9e9df1c 100644 --- a/tests/ShopBundle/Functional/Model/Payment/IndependentPaymentVisibilityCalculationTest.php +++ b/tests/ShopBundle/Functional/Model/Payment/IndependentPaymentVisibilityCalculationTest.php @@ -114,7 +114,7 @@ public function testIsIndependentlyVisibleHidden() * @param \Shopsys\FrameworkBundle\Model\Pricing\Vat\Vat $vat * @param bool[] $enabledForDomains * @param bool $hidden - * @return Payment + * @return \Shopsys\FrameworkBundle\Model\Payment\Payment */ public function getDefaultPayment(Vat $vat, $enabledForDomains, $hidden) { diff --git a/tests/ShopBundle/Functional/Model/Pricing/InputPriceRecalculationSchedulerTest.php b/tests/ShopBundle/Functional/Model/Pricing/InputPriceRecalculationSchedulerTest.php index dff76d832..0c4b7dc00 100644 --- a/tests/ShopBundle/Functional/Model/Pricing/InputPriceRecalculationSchedulerTest.php +++ b/tests/ShopBundle/Functional/Model/Pricing/InputPriceRecalculationSchedulerTest.php @@ -88,6 +88,9 @@ private function createProductWithInputPriceAndVatPercentAndAutoCalculationPrice /** * @dataProvider inputPricesTestDataProvider + * @param mixed $inputPriceWithoutVat + * @param mixed $inputPriceWithVat + * @param mixed $vatPercent */ public function testOnKernelResponseRecalculateInputPricesWithoutVat( $inputPriceWithoutVat, @@ -166,6 +169,9 @@ public function testOnKernelResponseRecalculateInputPricesWithoutVat( /** * @dataProvider inputPricesTestDataProvider + * @param mixed $inputPriceWithoutVat + * @param mixed $inputPriceWithVat + * @param mixed $vatPercent */ public function testOnKernelResponseRecalculateInputPricesWithVat( $inputPriceWithoutVat, diff --git a/tests/ShopBundle/Functional/Model/Product/Availability/ProductAvailabilityCalculationTest.php b/tests/ShopBundle/Functional/Model/Product/Availability/ProductAvailabilityCalculationTest.php index 7a58b9c03..de1d64d0c 100644 --- a/tests/ShopBundle/Functional/Model/Product/Availability/ProductAvailabilityCalculationTest.php +++ b/tests/ShopBundle/Functional/Model/Product/Availability/ProductAvailabilityCalculationTest.php @@ -18,6 +18,13 @@ class ProductAvailabilityCalculationTest extends FunctionalTestCase { /** * @dataProvider getTestCalculateAvailabilityData + * @param mixed $usingStock + * @param mixed $stockQuantity + * @param mixed $outOfStockAction + * @param \Shopsys\FrameworkBundle\Model\Product\Availability\Availability|null $availability + * @param \Shopsys\FrameworkBundle\Model\Product\Availability\Availability|null $outOfStockAvailability + * @param \Shopsys\FrameworkBundle\Model\Product\Availability\Availability|null $defaultInStockAvailability + * @param \Shopsys\FrameworkBundle\Model\Product\Availability\Availability|null $expectedCalculatedAvailability */ public function testCalculateAvailability( $usingStock, diff --git a/tests/ShopBundle/Functional/Model/Product/ProductFacadeTest.php b/tests/ShopBundle/Functional/Model/Product/ProductFacadeTest.php index 883f1dd10..aac599d08 100644 --- a/tests/ShopBundle/Functional/Model/Product/ProductFacadeTest.php +++ b/tests/ShopBundle/Functional/Model/Product/ProductFacadeTest.php @@ -16,6 +16,12 @@ class ProductFacadeTest extends TransactionFunctionalTestCase { /** * @dataProvider getTestHandleOutOfStockStateDataProvider + * @param mixed $hidden + * @param mixed $sellingDenied + * @param mixed $stockQuantity + * @param mixed $outOfStockAction + * @param mixed $calculatedHidden + * @param mixed $calculatedSellingDenied */ public function testHandleOutOfStockState( $hidden, diff --git a/tests/ShopBundle/Functional/Model/Product/ProductRepositoryTest.php b/tests/ShopBundle/Functional/Model/Product/ProductRepositoryTest.php index 7380a171a..a9c800a1e 100644 --- a/tests/ShopBundle/Functional/Model/Product/ProductRepositoryTest.php +++ b/tests/ShopBundle/Functional/Model/Product/ProductRepositoryTest.php @@ -37,6 +37,10 @@ public function testProductMainVariantIsListed() $this->getAllListableQueryBuilderTest(148, true); } + /** + * @param mixed $productReferenceId + * @param mixed $isExpectedInResult + */ private function getAllListableQueryBuilderTest($productReferenceId, $isExpectedInResult) { $productRepository = $this->getContainer()->get(ProductRepository::class); @@ -78,6 +82,10 @@ public function testProductMainVariantIsNotSellable() $this->getAllSellableQueryBuilderTest(148, false); } + /** + * @param mixed $productReferenceId + * @param mixed $isExpectedInResult + */ private function getAllSellableQueryBuilderTest($productReferenceId, $isExpectedInResult) { $productRepository = $this->getContainer()->get(ProductRepository::class); @@ -119,6 +127,10 @@ public function testProductMainVariantIsOfferred() $this->getAllOfferedQueryBuilderTest(69, true); } + /** + * @param mixed $productReferenceId + * @param mixed $isExpectedInResult + */ private function getAllOfferedQueryBuilderTest($productReferenceId, $isExpectedInResult) { $productRepository = $this->getContainer()->get(ProductRepository::class); diff --git a/tests/ShopBundle/Functional/Twig/DateTimeFormatterExtensionTest.php b/tests/ShopBundle/Functional/Twig/DateTimeFormatterExtensionTest.php index b5a0d1672..745803f2c 100644 --- a/tests/ShopBundle/Functional/Twig/DateTimeFormatterExtensionTest.php +++ b/tests/ShopBundle/Functional/Twig/DateTimeFormatterExtensionTest.php @@ -23,6 +23,9 @@ public function formatDateDataProvider() /** * @dataProvider formatDateDataProvider + * @param mixed $input + * @param mixed $locale + * @param mixed $result */ public function testFormatDate($input, $locale, $result) { diff --git a/tests/ShopBundle/Functional/Twig/NumberFormatterExtensionTest.php b/tests/ShopBundle/Functional/Twig/NumberFormatterExtensionTest.php index 96d7094e9..ff68d7662 100644 --- a/tests/ShopBundle/Functional/Twig/NumberFormatterExtensionTest.php +++ b/tests/ShopBundle/Functional/Twig/NumberFormatterExtensionTest.php @@ -34,6 +34,9 @@ public function formatNumberDataProvider() /** * @dataProvider formatNumberDataProvider + * @param mixed $input + * @param mixed $locale + * @param mixed $result */ public function testFormatNumber($input, $locale, $result) { diff --git a/tests/ShopBundle/Functional/Twig/PriceExtensionTest.php b/tests/ShopBundle/Functional/Twig/PriceExtensionTest.php index a343b5e45..4f667b16a 100644 --- a/tests/ShopBundle/Functional/Twig/PriceExtensionTest.php +++ b/tests/ShopBundle/Functional/Twig/PriceExtensionTest.php @@ -93,6 +93,9 @@ public function priceFilterDataProviderMultiDomain() /** * @group singledomain * @dataProvider priceFilterDataProviderSingleDomain + * @param mixed $input + * @param mixed $domainId + * @param mixed $result */ public function testPriceFilterForSingleDomain($input, $domainId, $result) { @@ -102,6 +105,9 @@ public function testPriceFilterForSingleDomain($input, $domainId, $result) /** * @group multidomain * @dataProvider priceFilterDataProviderMultiDomain + * @param mixed $input + * @param mixed $domainId + * @param mixed $result */ public function testPriceFilterForMultiDomain($input, $domainId, $result) { @@ -110,6 +116,9 @@ public function testPriceFilterForMultiDomain($input, $domainId, $result) /** * @dataProvider priceFilterDataProvider + * @param mixed $input + * @param mixed $domainId + * @param mixed $result */ private function checkPriceFilter($input, $domainId, $result) { diff --git a/tests/ShopBundle/Performance/Feed/PerformanceResultsCsvExporter.php b/tests/ShopBundle/Performance/Feed/PerformanceResultsCsvExporter.php index b23311a33..13c094350 100644 --- a/tests/ShopBundle/Performance/Feed/PerformanceResultsCsvExporter.php +++ b/tests/ShopBundle/Performance/Feed/PerformanceResultsCsvExporter.php @@ -11,6 +11,9 @@ class PerformanceResultsCsvExporter */ private $jmeterCsvReporter; + /** + * @param \Tests\ShopBundle\Performance\JmeterCsvReporter $jmeterCsvReporter + */ public function __construct(JmeterCsvReporter $jmeterCsvReporter) { $this->jmeterCsvReporter = $jmeterCsvReporter; diff --git a/tests/ShopBundle/Performance/Page/PerformanceResultsCsvExporter.php b/tests/ShopBundle/Performance/Page/PerformanceResultsCsvExporter.php index 592ba3220..a8ad3e3ee 100644 --- a/tests/ShopBundle/Performance/Page/PerformanceResultsCsvExporter.php +++ b/tests/ShopBundle/Performance/Page/PerformanceResultsCsvExporter.php @@ -11,6 +11,9 @@ class PerformanceResultsCsvExporter */ private $jmeterCsvReporter; + /** + * @param \Tests\ShopBundle\Performance\JmeterCsvReporter $jmeterCsvReporter + */ public function __construct(JmeterCsvReporter $jmeterCsvReporter) { $this->jmeterCsvReporter = $jmeterCsvReporter; diff --git a/tests/ShopBundle/Performance/Page/PerformanceTestSummaryPrinter.php b/tests/ShopBundle/Performance/Page/PerformanceTestSummaryPrinter.php index c72f48c16..b8bf37bb1 100644 --- a/tests/ShopBundle/Performance/Page/PerformanceTestSummaryPrinter.php +++ b/tests/ShopBundle/Performance/Page/PerformanceTestSummaryPrinter.php @@ -11,6 +11,9 @@ class PerformanceTestSummaryPrinter */ private $performanceTestSampleQualifier; + /** + * @param \Tests\ShopBundle\Performance\Page\PerformanceTestSampleQualifier $performanceTestSampleQualifier + */ public function __construct(PerformanceTestSampleQualifier $performanceTestSampleQualifier) { $this->performanceTestSampleQualifier = $performanceTestSampleQualifier; diff --git a/tests/ShopBundle/Smoke/NewProductTest.php b/tests/ShopBundle/Smoke/NewProductTest.php index e3e5a90bc..1bfb9a901 100644 --- a/tests/ShopBundle/Smoke/NewProductTest.php +++ b/tests/ShopBundle/Smoke/NewProductTest.php @@ -19,6 +19,7 @@ public function createOrEditProductProvider() /** * @dataProvider createOrEditProductProvider + * @param mixed $relativeUrl */ public function testCreateOrEditProduct($relativeUrl) { diff --git a/tests/ShopBundle/Test/Codeception/Module/StrictWebDriver.php b/tests/ShopBundle/Test/Codeception/Module/StrictWebDriver.php index 5572e7879..e8bfdc9ea 100644 --- a/tests/ShopBundle/Test/Codeception/Module/StrictWebDriver.php +++ b/tests/ShopBundle/Test/Codeception/Module/StrictWebDriver.php @@ -55,6 +55,8 @@ protected function findFields($selector) /** * @deprecated + * @param mixed $link + * @param mixed|null $context */ public function click($link, $context = null) { @@ -67,6 +69,8 @@ public function click($link, $context = null) /** * @see click() + * @param mixed $link + * @param mixed|null $context */ private function clickAndWait($link, $context = null) { @@ -129,6 +133,8 @@ public function clickByElement(WebDriverElement $element) /** * @deprecated + * @param mixed $field + * @param mixed $value */ public function fillField($field, $value) { @@ -190,6 +196,7 @@ public function seeInElement($text, WebDriverElement $element) /** * @deprecated + * @param mixed $checkbox */ public function seeCheckboxIsChecked($checkbox) { @@ -236,6 +243,7 @@ public function seeCheckboxIsCheckedByLabel($label) /** * @deprecated + * @param mixed $checkbox */ public function dontSeeCheckboxIsChecked($checkbox) { @@ -282,6 +290,7 @@ public function dontSeeCheckboxIsCheckedByLabel($label) /** * @deprecated + * @param mixed $option */ public function checkOption($option) { @@ -347,6 +356,8 @@ public function countVisibleByCss($css) /** * @deprecated + * @param mixed $field + * @param mixed $value */ public function seeInField($field, $value) { @@ -390,6 +401,8 @@ public function moveMouseOverByCss($css, $offsetX = null, $offsetY = null) /** * @deprecated + * @param mixed $element + * @param mixed $char */ public function pressKey($element, $char) {