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

hotfix: add an exception for item per page equal 0 #1141

Conversation

Simperfit
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #1118
License MIT

dunglas and others added 30 commits December 1, 2016 13:47
Make filters's ids defaulting to the service name
Add configuration option to define custom loader paths
Use the stable version of PHP CS Fixer 2
Make swagger UI compatible in a CSP environment
…-comparison

Add nulls comparison to OrderFilter
…fier

Move normalizeIdentifiers method to a new Util class IdentifierManager
…test

minor: add missing test for contextBuilder
Move properties in IdentifierManagerTrait and fix CS.
@grachevko
Copy link

grachevko commented May 26, 2017

What if I send negative number? :)

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '-1 OFFSET 0' at line 1"

@Simperfit Simperfit force-pushed the hotfix/add-an-exception-for-item-per-page branch from 6ac81e5 to 83a599a Compare May 26, 2017 11:59
Copy link
Member

@soyuka soyuka left a comment

Choose a reason for hiding this comment

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

Just run php-cs-fixer on the test file :) Then 👍

@Simperfit Simperfit force-pushed the hotfix/add-an-exception-for-item-per-page branch from 3692352 to a9dc54a Compare May 29, 2017 11:39
@Simperfit
Copy link
Contributor Author

@soyuka done :)

@soyuka
Copy link
Member

soyuka commented May 29, 2017

bug fix no? Shouldn't this target 2.0?

@Simperfit Simperfit changed the base branch from master to 2.0 May 29, 2017 12:51
@Simperfit Simperfit force-pushed the hotfix/add-an-exception-for-item-per-page branch from a9dc54a to 8eb4e65 Compare May 29, 2017 12:54
@dunglas
Copy link
Member

dunglas commented May 29, 2017

@Simperfit you need to rebase ;)

@Simperfit Simperfit closed this May 29, 2017
@Simperfit Simperfit deleted the hotfix/add-an-exception-for-item-per-page branch May 29, 2017 13:03
@Simperfit Simperfit restored the hotfix/add-an-exception-for-item-per-page branch May 29, 2017 13:03
@Simperfit Simperfit reopened this May 29, 2017
@Simperfit Simperfit closed this May 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.