Conversation
fe9e096
to
ff148bb
Compare
inc/deploypackage.class.php
Outdated
$packages_used = $this->getMyDepoyPackages($my_packages, $users_id); | ||
|
||
$packages = $this->canUserDeploySelf(); | ||
//Get packages that a the user can deploy | ||
$packages = $this->canUserDeploySelf(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clear spaces before '=', i don't see any lines where this alignment is needed
inc/deploypackage.class.php
Outdated
'name' => '[deploy on demand] '.$this->fields['name'], | ||
'entities_id' => $computer->fields['entities_id'], | ||
'reprepare_if_successful' => 0, | ||
'is_deploy_on_demand' => 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alignement
inc/deploypackage.class.php
Outdated
); | ||
|
||
//Enable the task | ||
$input = ['id' => $tasks_id, 'is_active' => 1]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not activating the task on the previous "->add($input)"
$a_menu[6]['link'] = $CFG_GLPI['root_doc']. | ||
"/plugins/fusioninventory/front/ruledictionnarycomputerarch.php"; | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You removed computer architectures rules from menu ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, not needed anymore as it's been transfered into glpi core since 9.1
inc/task.class.php
Outdated
return array('description' => __('FusionInventory task scheduler')); | ||
|
||
case 'cleanOnDemand' : | ||
return array('description' => __('Clean on deman deployment tasks')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo, deman -> demand
remove fi at ant eol
…ed. Update unit tests
5c07a85
to
67e9b05
Compare
* Add right for one time deployment * Remove unused code & menus * Work on generization * Factorize code * fActorize code * Continue rename Deploy on demand task * Add an option to force a local or remote agent wakeup * Add an option to not wakeup the agent * Add crontasj to clean on demand tasks if needed * Split crontask in two to be able to test it * Fix unit tests * Update on demande deploy tests * wait a little time for local wakeup * More understandable wakeup messages * remove coverage for php7.1 remove fi at ant eol * Do not propose packages to deploy when agent has no deploy task enabled. Update unit tests * fix id used in package edition (we change package not orders) should fix #2019 * fix tests fix tests fix tests fix tests CS tests fix tests
Ability to deploy packages on demand, not only for a self service person.
Add informations about packages available and being installed in the task tab of a computer.
Also add the possiblity, when user is using the standard interface to select if wakeup should be local (calling the agent as 127.0.0.1) or remote, from the server.