Some fixes

[marcelklehr]

see floccusaddon/floccus#1983

[Copilot]

Pull Request Overview

This PR enhances type safety for database queries by importing IQueryBuilder and specifying IQueryBuilder::PARAM_INT for positional and named parameters across migrations and mappers. It also improves error handling in TreeMapper::getSubFolders by catching loading exceptions and filtering out failed shares.

• Imported IQueryBuilder and added PARAM_INT to all createPositionalParameter/createNamedParameter calls
• Added try/catch around shared-folder loading in TreeMapper and filtered out null entries

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
lib/Migration/Version003000000Date20191123094721.php	Added IQueryBuilder import and PARAM_INT for positional parameters in postSchemaChange
lib/Migration/Version000014000Date20181029094721.php	Added IQueryBuilder import and PARAM_INT in postSchemaChange
lib/Migration/OrphanedSharesRepairStep.php	Added IQueryBuilder import and PARAM_INT for delete queries
lib/Db/TreeMapper.php	Specified PARAM_INT in move, added exception handling and filtered null in getSubFolders
lib/Db/TagMapper.php	Added IQueryBuilder import and PARAM_INT for tag lookups
lib/Db/SharedFolderMapper.php	Added IQueryBuilder import and PARAM_INT for shared-folder queries
lib/Db/ShareMapper.php	Added IQueryBuilder import and PARAM_INT for share queries and annotated exceptions
lib/Db/PublicFolderMapper.php	Added IQueryBuilder import and PARAM_INT for named parameters
lib/Db/FolderMapper.php	Added IQueryBuilder import and PARAM_INT for find and insert operations
lib/Db/BookmarkMapper.php	Added IQueryBuilder import and PARAM_INT for bookmark queries

Comments suppressed due to low confidence (2)

lib/Db/SharedFolderMapper.php:222

• There's a typo in the constant name PARAM_INTy. It should be IQueryBuilder::PARAM_INT.

			->andWhere($qb->expr()->eq('s.type', $qb->createPositionalParameter($type, IQueryBuilder::PARAM_INTy)))

lib/Db/ShareMapper.php:51

• There is an extra semicolon (;;) at the end of this line. Removing the duplicate semicolon will resolve the syntax error.

			->where($qb->expr()->eq('id', $qb->createPositionalParameter($shareId, IQueryBuilder::PARAM_INT)));;