Skip to content

Conversation

@sinri
Copy link
Contributor

@sinri sinri commented Jan 16, 2019

Make the following SQL parsable, which is mentioned in issue #221

truncate table a.b;truncate c;select truncate from truncate;

@codecov
Copy link

codecov bot commented Jan 16, 2019

Codecov Report

Merging #222 into master will decrease coverage by 0.04%.
The diff coverage is 75%.

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
- Coverage      99.7%   99.65%   -0.05%     
- Complexity     1841     1843       +2     
============================================
  Files            61       62       +1     
  Lines          4088     4091       +3     
============================================
+ Hits           4076     4077       +1     
- Misses           12       14       +2

@codecov
Copy link

codecov bot commented Jan 16, 2019

Codecov Report

Merging #222 into master will increase coverage by 0.06%.
The diff coverage is 75%.

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
+ Coverage     99.58%   99.65%   +0.06%     
+ Complexity     1856     1843      -13     
============================================
  Files            62       62              
  Lines          4127     4091      -36     
============================================
- Hits           4110     4077      -33     
+ Misses           17       14       -3

Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

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

LGTM

@sinri
Copy link
Contributor Author

sinri commented Jan 17, 2019

Wonder why conflicts came out.

@sinri
Copy link
Contributor Author

sinri commented Jan 17, 2019

Merged the latest master branch to resolve conflicts.

@devenbansod devenbansod self-requested a review January 20, 2019 15:30
@stale
Copy link

stale bot commented Mar 21, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Mar 21, 2019
Copy link
Member

@ibennetch ibennetch left a comment

Choose a reason for hiding this comment

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

@devenbansod would you have a moment to take a quick look and see if this seems suitable to you?

It looks great to me aside from the possible debugging function I mentioned.

src/Token.php Outdated
}

/**
* Make you happier when you need to debug.
Copy link
Member

Choose a reason for hiding this comment

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

Is this function valuable to include in the code or is it debugging code that should be removed?

If it's worth keeping, I think the comment here should be changed to make the purpose of the function more clear.

@ibennetch ibennetch changed the title Issue 221 Fix issue with parsing TRUNCATE statements, Issue #221 Apr 7, 2019
@stale
Copy link

stale bot commented Jun 6, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jun 6, 2019
@stale stale bot closed this Jun 13, 2019
@williamdes williamdes reopened this Jun 13, 2019
@stale stale bot removed the wontfix label Jun 13, 2019
@stale
Copy link

stale bot commented Aug 12, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Aug 12, 2019
@stale
Copy link

stale bot commented Oct 11, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Oct 11, 2019
Fixes: phpmyadmin#221
Pull-request: phpmyadmin#222

Co-authored-by: Sinri Edogawa <e.joshua.s.e@gmail.com>
Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes williamdes changed the base branch from master to QA October 28, 2019 22:55
williamdes added a commit that referenced this pull request Oct 28, 2019
Fixes: #221
Pull-request: #222

Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes williamdes merged commit 04da59b into phpmyadmin:QA Oct 28, 2019
williamdes added a commit that referenced this pull request Oct 28, 2019
Pull-request: #222
Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes
Copy link
Member

Merged as 04da59b
Thanks for your contribution and patience @sinri !

@williamdes williamdes self-assigned this Oct 28, 2019
@williamdes williamdes added this to the 4.3.3 milestone Oct 28, 2019
@sinri
Copy link
Contributor Author

sinri commented Oct 30, 2019

Great Result Finally!

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