Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[framework] private -> protected in Shopsys\FrameworkBundle\Twig namespace #1114

Merged
merged 1 commit into from Jun 17, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions packages/framework/easy-coding-standard.yml
Expand Up @@ -12,6 +12,7 @@ services:
- Shopsys\FrameworkBundle\Model
- Shopsys\FrameworkBundle\Component
- Shopsys\FrameworkBundle\Controller
- Shopsys\FrameworkBundle\Twig

Shopsys\CodingStandards\Sniffs\ConstantVisibilityRequiredSniff: ~

Expand Down
2 changes: 1 addition & 1 deletion packages/framework/src/Twig/CookiesExtension.php
Expand Up @@ -10,7 +10,7 @@ class CookiesExtension extends \Twig_Extension
/**
* @var \Shopsys\FrameworkBundle\Model\Cookies\CookiesFacade
*/
private $cookiesFacade;
protected $cookiesFacade;

/**
* @param \Shopsys\FrameworkBundle\Model\Cookies\CookiesFacade $cookiesFacade
Expand Down
2 changes: 1 addition & 1 deletion packages/framework/src/Twig/CssExtension.php
Expand Up @@ -10,7 +10,7 @@ class CssExtension extends \Twig_Extension
/**
* @var \Shopsys\FrameworkBundle\Component\Css\CssFacade
*/
private $cssFacade;
protected $cssFacade;

/**
* @param \Shopsys\FrameworkBundle\Component\Css\CssFacade $cssFacade
Expand Down
10 changes: 5 additions & 5 deletions packages/framework/src/Twig/DateTimeFormatterExtension.php
Expand Up @@ -14,12 +14,12 @@ class DateTimeFormatterExtension extends Twig_Extension
/**
* @var \Shopsys\FrameworkBundle\Component\Localization\DateTimeFormatter
*/
private $dateTimeFormatter;
protected $dateTimeFormatter;

/**
* @var \Shopsys\FrameworkBundle\Model\Localization\Localization
*/
private $localization;
protected $localization;

/**
* @param \Shopsys\FrameworkBundle\Component\Localization\DateTimeFormatter $dateTimeFormatter
Expand Down Expand Up @@ -119,7 +119,7 @@ public function formatDateTime($dateTime, $locale = null)
* @param string|null $locale
* @return string
*/
private function format($dateTime, $dateType, $timeType, $locale = null)
protected function format($dateTime, $dateType, $timeType, $locale = null)
{
if ($dateTime === null) {
return '-';
Expand All @@ -137,7 +137,7 @@ private function format($dateTime, $dateType, $timeType, $locale = null)
* @param string|null $locale
* @return string
*/
private function getLocale($locale = null)
protected function getLocale($locale = null)
{
if ($locale === null) {
$locale = $this->localization->getLocale();
Expand All @@ -150,7 +150,7 @@ private function getLocale($locale = null)
* @param mixed $value
* @return \DateTime
*/
private function convertToDateTime($value)
protected function convertToDateTime($value)
{
if ($value instanceof DateTime) {
return $value;
Expand Down
8 changes: 4 additions & 4 deletions packages/framework/src/Twig/DomainExtension.php
Expand Up @@ -12,22 +12,22 @@ class DomainExtension extends \Twig_Extension
/**
* @var string
*/
private $domainImagesUrlPrefix;
protected $domainImagesUrlPrefix;

/**
* @var \Symfony\Component\Asset\Packages
*/
private $assetPackages;
protected $assetPackages;

/**
* @var \Shopsys\FrameworkBundle\Component\Domain\Domain
*/
private $domain;
protected $domain;

/**
* @var \Shopsys\FrameworkBundle\Component\Domain\DomainFacade
*/
private $domainFacade;
protected $domainFacade;

/**
* @param mixed $domainImagesUrlPrefix
Expand Down
Expand Up @@ -16,17 +16,17 @@ class FileThumbnailExtension extends Twig_Extension
/**
* @var string[]
*/
private $iconsByExtension;
protected $iconsByExtension;

/**
* @var \Shopsys\FrameworkBundle\Component\FileUpload\FileUpload
*/
private $fileUpload;
protected $fileUpload;

/**
* @var \Shopsys\FrameworkBundle\Component\Image\Processing\ImageThumbnailFactory
*/
private $imageThumbnailFactory;
protected $imageThumbnailFactory;

/**
* @param \Shopsys\FrameworkBundle\Component\FileUpload\FileUpload $fileUpload
Expand Down Expand Up @@ -98,7 +98,7 @@ public function getFileThumbnailInfoByTemporaryFilename($temporaryFilename)
* @param string $filepath
* @return \Shopsys\FrameworkBundle\Twig\FileThumbnail\FileThumbnailInfo
*/
private function getImageThumbnailInfo($filepath)
protected function getImageThumbnailInfo($filepath)
{
$image = $this->imageThumbnailFactory->getImageThumbnail($filepath);

Expand All @@ -109,7 +109,7 @@ private function getImageThumbnailInfo($filepath)
* @param string $filepath
* @return string
*/
private function getIconTypeByFilename($filepath)
protected function getIconTypeByFilename($filepath)
{
$extension = strtolower(pathinfo($filepath, PATHINFO_EXTENSION));
if (array_key_exists($extension, $this->iconsByExtension)) {
Expand Down
Expand Up @@ -7,12 +7,12 @@ class FileThumbnailInfo
/**
* @var string|null
*/
private $iconType;
protected $iconType;

/**
* @var string|null
*/
private $imageUri;
protected $imageUri;

/**
* @param string|null $iconType
Expand Down
2 changes: 1 addition & 1 deletion packages/framework/src/Twig/FileUploadExtension.php
Expand Up @@ -11,7 +11,7 @@ class FileUploadExtension extends Twig_Extension
/**
* @var \Shopsys\FrameworkBundle\Component\FileUpload\FileUpload
*/
private $fileUpload;
protected $fileUpload;

/**
* @param \Shopsys\FrameworkBundle\Component\FileUpload\FileUpload $fileUpload
Expand Down
2 changes: 1 addition & 1 deletion packages/framework/src/Twig/FormDetailExtension.php
Expand Up @@ -12,7 +12,7 @@ class FormDetailExtension extends Twig_Extension
/**
* @var \Twig_Environment
*/
private $twigEnvironment;
protected $twigEnvironment;

/**
* @param \Twig_Environment $twigEnvironment
Expand Down
2 changes: 1 addition & 1 deletion packages/framework/src/Twig/FormThemeExtension.php
Expand Up @@ -54,7 +54,7 @@ public function getDefaultFormTheme()
* @param string $controller
* @return bool
*/
private function isAdmin(string $controller) : bool
protected function isAdmin(string $controller) : bool
{
return strpos($controller, 'Shopsys\FrameworkBundle\Controller\Admin') === 0 ||
strpos($controller, 'Shopsys\ShopBundle\Controller\Admin') === 0;
Expand Down
4 changes: 2 additions & 2 deletions packages/framework/src/Twig/HoneyPotExtension.php
Expand Up @@ -48,7 +48,7 @@ public function formHoneyPotCheckPasswordAlreadyRendered(FormView $formView)
* @param \Symfony\Component\Form\FormView $formView
* @return \Symfony\Component\Form\FormView
*/
private function getRootFormView(FormView $formView)
protected function getRootFormView(FormView $formView)
{
$rootFormView = $formView;

Expand All @@ -63,7 +63,7 @@ private function getRootFormView(FormView $formView)
* @param \Symfony\Component\Form\FormView $formView
* @return bool
*/
private function containsNotRenderedPassword(FormView $formView)
protected function containsNotRenderedPassword(FormView $formView)
{
foreach ($formView->children as $childForm) {
if (strpos($childForm->vars['name'], static::PASSWORD_FIELD_NAME) !== false && !$childForm->isRendered()) {
Expand Down
10 changes: 5 additions & 5 deletions packages/framework/src/Twig/ImageExtension.php
Expand Up @@ -20,7 +20,7 @@ class ImageExtension extends Twig_Extension
/**
* @var string
*/
private $frontDesignImageUrlPrefix;
protected $frontDesignImageUrlPrefix;

/**
* @var \Shopsys\FrameworkBundle\Component\Domain\Domain
Expand All @@ -30,7 +30,7 @@ class ImageExtension extends Twig_Extension
/**
* @var \Shopsys\FrameworkBundle\Component\Image\ImageLocator
*/
private $imageLocator;
protected $imageLocator;

/**
* @var \Shopsys\FrameworkBundle\Component\Image\ImageFacade
Expand All @@ -40,7 +40,7 @@ class ImageExtension extends Twig_Extension
/**
* @var \Symfony\Component\Templating\EngineInterface
*/
private $templating;
protected $templating;

/**
* @param string $frontDesignImageUrlPrefix
Expand Down Expand Up @@ -157,7 +157,7 @@ public function getNoimageHtml(array $attributes = [])
/**
* @return string
*/
private function getEmptyImageUrl(): string
protected function getEmptyImageUrl(): string
{
return $this->domain->getUrl() . $this->frontDesignImageUrlPrefix . '/' . static::NOIMAGE_FILENAME;
}
Expand All @@ -168,7 +168,7 @@ private function getEmptyImageUrl(): string
* @param string|null $sizeName
* @return string
*/
private function getImageCssClass($entityName, $type, $sizeName)
protected function getImageCssClass($entityName, $type, $sizeName)
{
$allClassParts = [
'image',
Expand Down
2 changes: 1 addition & 1 deletion packages/framework/src/Twig/InputPriceLabelExtension.php
Expand Up @@ -11,7 +11,7 @@ class InputPriceLabelExtension extends Twig_Extension
/**
* @var \Shopsys\FrameworkBundle\Model\Pricing\PricingSetting
*/
private $pricingSetting;
protected $pricingSetting;

/**
* @param \Shopsys\FrameworkBundle\Model\Pricing\PricingSetting $pricingSetting
Expand Down