FIX Don't generate table alias for "from" statement that are not column names

[maxime-rainville]

Manual testing steps

This is updating a low level API that is widely use across tho codebase. There's not really any manual steps that could be taken ... aside from using the system.

Issues

• SQLSelect regression with subqueries #11276

Pull request checklist

• The target branch is correct
• All commits are relevant to the purpose of the PR (e.g. no debug statements, unrelated refactoring, or arbitrary linting)
  • Small amounts of additional linting are usually okay, but if it makes it hard to concentrate on the relevant changes, ask for the unrelated changes to be reverted, and submitted as a separate PR.
• The commit messages follow our commit message guidelines
• The PR follows our contribution guidelines
• Code changes follow our coding conventions
• This change is covered with tests (or tests aren't necessary for this change)
• Any relevant User Help/Developer documentation is updated; for impactful changes, information is added to the changelog for the intended release
• CI is green

[maxime-rainville]

That's the only scenario that would have failed prior to this PR. The other ones are just there for the sake of completeness.

[maxime-rainville]

Most of those would have pass previously, but addFrom didn't have explicit test coverage.

[maxime-rainville]

Got a sink build here with this PR: https://github.com/archiprocode/recipe-kitchen-sink/actions/runs/10103439804

The Lining failure is pre-existing.

[GuySartorelli]

I'd love to have $alias as a second parameter, so aliases have to be explicitly added instead of the system guessing what it should be.
But that would have to be a major release change so obviously isn't appropriate for this PR.

Seems like a good solution. Just a couple small changes to make.

[maxime-rainville]

I'll have implement the requested change tomorrow.

[GuySartorelli]

LGTM, thanks for doing this.