Skip to content

[Form] Remove unused private method #17528

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

Merged
merged 1 commit into from
Jan 25, 2016

Conversation

paradajozsef
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? FormValidatorTest::testMissingConstraintIndex() failed, but not relevant
Fixed tickets #1234
License MIT
Doc PR -

In 3.0 there is no reference to the private indent method.

There was only one reference in 2.8, but it has been removed.

@paradajozsef
Copy link
Contributor Author

Maybe a csfixer could filter these unused methods.

@fabpot
Copy link
Member

fabpot commented Jan 25, 2016

Thank you @paradajozsef.

@fabpot fabpot merged commit 0fb1664 into symfony:3.0 Jan 25, 2016
fabpot added a commit that referenced this pull request Jan 25, 2016
This PR was merged into the 3.0 branch.

Discussion
----------

[Form] Remove unused private method

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | FormValidatorTest::testMissingConstraintIndex() failed, but not relevant
| Fixed tickets | #1234
| License       | MIT
| Doc PR        | -

In 3.0 there is no reference to the private [indent method](https://github.com/symfony/symfony/blob/3.0/src/Symfony/Component/Form/Form.php#L1149).

There was only [one reference in 2.8](https://github.com/symfony/symfony/blob/2.8/src/Symfony/Component/Form/Form.php#L843), but it has been removed.

Commits
-------

0fb1664 [Form] Remove unused private method
@paradajozsef paradajozsef deleted the form-remove-unused-method-3.0 branch January 25, 2016 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants