Skip to content

Fix parsing of INSERT...SELECT and INSERT...SET syntax #76

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 2 commits into from
Sep 13, 2016

Conversation

devenbansod
Copy link
Member

Implement separate parse function for InsertStatement

Fix #50
Fix #73
Fix phpmyadmin/phpmyadmin#11628

Signed-off-by: Deven Bansod devenbansod.bits@gmail.com

Implement separate parse function for InsertStatement

Fix phpmyadmin#50
Fix phpmyadmin#73
Fix phpmyadmin/phpmyadmin#11628

Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
@codecov-io
Copy link

codecov-io commented Sep 12, 2016

Current coverage is 96.98% (diff: 85.71%)

Merging #76 into master will decrease coverage by 0.27%

@@             master        #76   diff @@
==========================================
  Files            49         49          
  Lines          3145       3219    +74   
  Methods         165        166     +1   
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           3059       3122    +63   
- Misses           86         97    +11   
  Partials          0          0          

Powered by Codecov. Last update bd8936d...7e6fbf7

Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
@nijel nijel self-assigned this Sep 13, 2016
@nijel
Copy link
Contributor

nijel commented Sep 13, 2016

@nijel nijel merged commit e78cf11 into phpmyadmin:master Sep 13, 2016
nijel added a commit that referenced this pull request Sep 13, 2016
Issue #76

Signed-off-by: Michal Čihař <michal@cihar.com>
nijel added a commit that referenced this pull request Sep 13, 2016
Issue #76

Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel
Copy link
Contributor

nijel commented Sep 13, 2016

Added two more tests:

  • 98c2a72 Add test for building INSERT ... SELECT ...
  • c7cc3ed Add test for building INSERT ... SET.

While trying to add third test, I've noticed that the ON DUPLICATE handling is still not working as expected, see #78.

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.

3 participants