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

IBX-154: Allowed to define if the alternative text for image field is required (2.5) #3108

Merged
merged 2 commits into from
Jun 25, 2021

Conversation

adamwojs
Copy link
Member

Question Answer
JIRA issue IBX-154
Bug/Improvement yes
New feature no
Target version 7.5
BC breaks no
Tests pass yes
Doc needed no

ezsystems/ezplatform-kernel#194 rebased to 2.5

TODO:

  • Implement feature / fix a bug.
  • Implement tests.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@adamwojs adamwojs self-assigned this Jun 16, 2021
Comment on lines +24 to +28
/** @var \eZ\Publish\Core\IO\UrlRedecoratorInterface|\PHPUnit\Framework\MockObject\MockObject */
private $urlRedecorator;

/** @var \eZ\Publish\Core\IO\IOServiceInterface|\PHPUnit\Framework\MockObject\MockObject */
private $ioService;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that MockObject can be dropped from those for the purposes of future phpstan introduction - they are not configured anywhere.

@piotrbisaga piotrbisaga self-assigned this Jun 21, 2021
Copy link

@piotrbisaga piotrbisaga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA approved, checked on eZ Platform Enterprise 2.5 with .diff

@piotrbisaga piotrbisaga removed their assignment Jun 24, 2021
@adamwojs adamwojs merged commit a98938b into 7.5 Jun 25, 2021
@adamwojs adamwojs deleted the ibx_154_25 branch June 25, 2021 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants