Add sync option to execute form
2 unresolved threads
2 unresolved threads
Closes #3443572
Merge request reports
Activity
143 143 '#description' => $this->t('Check this box to update all previously-imported content in addition to importing new content. Leave unchecked to only import new content'), 144 144 ]; 145 145 146 $form['options']['sync'] = [ 147 '#type' => 'checkbox', 148 '#title' => $this->t('Sync'), 149 '#description' => $this->t('Check this box to sync source and destination. Delete destination records that do not exist in the source.'), changed this line in version 2 of the diff
252 258 $options = [ 253 259 'limit' => $form_state->getValue('limit') ?: 0, 254 260 'update' => $form_state->getValue('update') ?: 0, 261 'sync' => (bool)$form_state->getValue('update'), Missing a space after the
(bool)
. Additionally,sync
was written asupdate
. Was the code tested before the MR was created?Edited by Jerry Radwickchanged this line in version 2 of the diff
added 1 commit
added 5 commits
-
1e35a528...332f3c65 - 3 commits from branch
project:6.0.x
- 58629598 - Add sync option to execute form
- 9d86dad9 - Issue #3443572: Fix "sync" written as "update"
-
1e35a528...332f3c65 - 3 commits from branch
Please register or sign in to reply