Update ITable.php to include comment

[Chartman123]

• Resolves: #

Summary

In Forms with NC35 the Psalm actions fail because comment is missing in the docblock for addColumn and modifyColumn:

https://github.com/nextcloud/forms/actions/runs/31592467991/job/94100460339?pr=3578

This adds the missing array key to the docblocks.

TODO

• ...

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)
• Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
• Milestone added for target branch/version (ex: 32.x for stable32)

AI (if applicable)

• The content of this PR was partly or fully generated using AI

[CarlSchwan]

Thanks!

[Chartman123]

@CarlSchwan thanks for the quick review :)

in the same error log, you can find that the function name for changing a column has changed between migrations for NC34 and NC35. Until now, we used ISchemaWrapper with getTable which in turn returned Doctrine\DBAL\Schema\Table with changeColumn().

Now with OCP\DB\Schema\ITable it's called modifyColumn().

[CarlSchwan]

@Chartman123 changeColumn has been deprecated for a long time.

technically it will still work because OC\DB\Schema\Table has a __call method that will forward the call, so I am not super inclined to add it back to the public interface

[welcome]

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

[Chartman123]

@Chartman123 changeColumn has been deprecated for a long time.

technically it will still work because OC\DB\Schema\Table has a __call method that will forward the call, so I am not super inclined to add it back to the public interface

ok, then we'll adjust all existing migrations to the correct calls :)

[susnux]

technically it will still work because OC\DB\Schema\Table has a __call method that will forward the call, so I am not super inclined to add it back to the public interface

But that will break all existing apps. Not hard but from linting.
We should add this back and mark it as deprecated, no?

[Chartman123]

changeColumn() is only used 10 times within Nextclouds Github org:

https://github.com/search?q=org%3Anextcloud+changeColumn+path%3A%2F%5Elib%5C%2FMigration%5C%2F%2F&type=code