feat: disable BaseBuilder::setAlias()

[kenjis]

Description
Since there is not yet a consensus to make this method a public API, it will be disabled when v4.3.0 is released.

See #6734 (comment)

Checklist:

• Securely signed commits
• [] Component(s) with PHPDoc blocks, only if necessary or adds value
• [] Unit testing, with >80% coverage
• User guide updated
• Conforms to style guide

[kenjis]

What's wrong?

There was 1 error:

1) CodeIgniter\Database\Live\UpdateTest::testUpdateBatchUpdateFieldsAndAlias
CodeIgniter\Database\Exceptions\DatabaseException: oci_execute(): ORA-30926: unable to get a stable set of rows in the source tables

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseConnection.php:632
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:1786
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:2427
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/Live/UpdateTest.php:400
phpvfscomposer:///home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/phpunit:97

Caused by
CodeIgniter\Database\Exceptions\DatabaseException: oci_execute(): ORA-30926: unable to get a stable set of rows in the source tables

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/OCI8/Connection.php:211
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseConnection.php:678
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseConnection.php:600
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:1786
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:2427
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/Live/UpdateTest.php:400
phpvfscomposer:///home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/phpunit:97

Caused by
ErrorException: oci_execute(): ORA-30926: unable to get a stable set of rows in the source tables

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/OCI8/Connection.php:199
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseConnection.php:678
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseConnection.php:600
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:1786
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:2427
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/Live/UpdateTest.php:400
phpvfscomposer:///home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/phpunit:97

https://github.com/codeigniter4/CodeIgniter4/actions/runs/3475428925/jobs/5810650672

[MGatner]

Not sure what those errors mean but it looks like you got it working. Let's see if we can hear back from @sclubricants, but I'm good with this PR if it expedited 4.3

[kenjis]

Yes, I fixed the bug. I don't know why only OCI8 did not work, but my code was obviously wrong.

[sclubricants]

I can understand not wanting to make thing public that aren't necessary. I do find it easier to code having the separate public method. The method chaining seems easier to remember than the parameters. It does make more logical sense to tie it to the setData() method though.. as the alias is sure to be tied to the dataset and not to something else. If in the future there is a need for multiple aliases then this might not work so well.

[MGatner]

@sclubricants I think the intent is to hide this now so we can finish discussing it without blocking the 4.3 release. I am looking to the Database Team to have most of that discussion, but I think this is a good temporary measure.