Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Convert command option defaults to strings
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
  • Loading branch information
ChristophWurst committed Jun 9, 2021
commit bf564e2a5a5b19f99d6dad94f099a757f5a044ff
4 changes: 2 additions & 2 deletions apps/user_ldap/lib/Command/Search.php
Original file line number Diff line number Diff line change
Expand Up @@ -73,14 +73,14 @@ protected function configure() {
null,
InputOption::VALUE_REQUIRED,
'The offset of the result set. Needs to be a multiple of limit. defaults to 0.',
0
'0'
)
->addOption(
'limit',
null,
InputOption::VALUE_REQUIRED,
'limit the results. 0 means no limit, defaults to 15',
15
'15'
)
;
}
Expand Down
4 changes: 2 additions & 2 deletions core/Command/Db/ConvertType.php
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ protected function configure() {
null,
InputOption::VALUE_REQUIRED,
'the maximum number of database rows to handle in a single query, bigger tables will be handled in chunks of this size. Lower this if the process runs out of memory during conversion.',
1000
'1000'
)
;
}
Expand Down Expand Up @@ -306,7 +306,7 @@ protected function copyTable(Connection $fromDB, Connection $toDB, Table $table,
return;
}

$chunkSize = $input->getOption('chunk-size');
$chunkSize = (int)$input->getOption('chunk-size');

$query = $fromDB->getQueryBuilder();
$query->automaticTablePrefix(false);
Expand Down
4 changes: 2 additions & 2 deletions core/Command/Group/ListCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -52,13 +52,13 @@ protected function configure() {
'l',
InputOption::VALUE_OPTIONAL,
'Number of groups to retrieve',
500
'500'
)->addOption(
'offset',
'o',
InputOption::VALUE_OPTIONAL,
'Offset for retrieving groups',
0
'0'
)->addOption(
'info',
'i',
Expand Down
4 changes: 2 additions & 2 deletions core/Command/User/ListCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,13 +60,13 @@ protected function configure() {
'l',
InputOption::VALUE_OPTIONAL,
'Number of users to retrieve',
500
'500'
)->addOption(
'offset',
'o',
InputOption::VALUE_OPTIONAL,
'Offset for retrieving users',
0
'0'
)->addOption(
'output',
null,
Expand Down