Skip to content

Conversation

devenbansod
Copy link
Member

Fix parsing of NATURAL JOIN, CROSS JOIN and related joins
Fix broken parsing of multiple JOINS in a single query.

Fix #93

Add tests to cover parsing of these joins.

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

Fix phpmyadmin#93

Add tests to cover parsing of these joins

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

codecov-io commented Oct 24, 2016

Current coverage is 99.47% (diff: 100%)

Merging #95 into master will increase coverage by <.01%

@@             master        #95   diff @@
==========================================
  Files            53         53          
  Lines          3614       3622     +8   
  Methods         176        176          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           3595       3603     +8   
  Misses           19         19          
  Partials          0          0          

Powered by Codecov. Last update 53787e2...2a046e8

Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
@nijel nijel self-assigned this Oct 25, 2016
@nijel nijel merged commit 3e2e737 into phpmyadmin:master Oct 25, 2016
@nijel
Copy link
Contributor

nijel commented Oct 25, 2016

Thanks, looks great!

nijel added a commit that referenced this pull request Oct 25, 2016
Signed-off-by: Michal Čihař <michal@cihar.com>
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