Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Change Component abstract class to an interface#365

Merged
MauricioFauth merged 1 commit intophpmyadmin:6.0.xphpmyadmin/sql-parser:6.0.xfrom
MauricioFauth:component-interfaceMauricioFauth/sql-parser:component-interfaceCopy head branch name to clipboard
Jan 3, 2022
Merged

Change Component abstract class to an interface#365
MauricioFauth merged 1 commit intophpmyadmin:6.0.xphpmyadmin/sql-parser:6.0.xfrom
MauricioFauth:component-interfaceMauricioFauth/sql-parser:component-interfaceCopy head branch name to clipboard

Conversation

@MauricioFauth
Copy link
Member

The PhpMyAdmin\SqlParser\Component abstract class has only two methods that are not implemented serving only as a contract. Changing it to an interface makes more sense.

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
@MauricioFauth MauricioFauth merged commit 9a5eaab into phpmyadmin:6.0.x Jan 3, 2022
@MauricioFauth MauricioFauth deleted the component-interface branch January 3, 2022 18:57
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.