FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Implement table aliasing and extent the builder to have multiple… by MDaskalovAtBE · Pull Request #110 · nilportugues/php-sql-query-builder · GitHub

Implement table aliasing and extent the builder to have multiple… - #110

Open
MDaskalovAtBE wants to merge 2 commits into
nilportugues:masterfrom
MDaskalovAtBE:feature/table-aliases
Open

Implement table aliasing and extent the builder to have multiple…#110
MDaskalovAtBE wants to merge 2 commits into
nilportugues:masterfrom
MDaskalovAtBE:feature/table-aliases

Conversation

Copy link
Copy Markdown

…joins with the same table name but different aliases

…s with the same table name but different aliases
Comment thread src/Manipulation/AbstractBaseQuery.php Outdated
public function setTable($table)
{
$this->table = (string) $table;
if ( ! (is_object($table) && is_a($table, Table::class))) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Expected 0 spaces after opening bracket; 1 found

Comment thread src/Manipulation/JoinQuery.php Outdated
$joinType = null
) {
if (!isset($this->joins[$table])) {
if ( !(is_object($table) && is_a($table, Table::class))) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Expected 0 spaces after opening bracket; 1 found

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL