NuKVM/xen-orchestra
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(xo-web/backup-ng/new): the compression default value should be false (#3280)

Browse Source 
See #2711
This commit is contained in:
badrAZ 2018-08-08 11:03:29 +02:00 committed by Pierre Donias
parent 0ca7b8539f
commit 09a381806b
1 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
packages/xo-web/src/xo-app/backup-ng/new/index.js
View File

@ -112,7 +112,7 @@ const createDoesRetentionExist = name => {
const getInitialState = () => ({ const getInitialState = () => ({
$pool: {}, $pool: {},
backupMode: false, backupMode: false,
compression: true, compression: undefined,
crMode: false, crMode: false,
deltaMode: false, deltaMode: false,
drMode: false, drMode: false,
@ -244,7 +244,12 @@ export default [
id: props.job.id, id: props.job.id,
name: state.name, name: state.name,
mode: state.isDelta ? 'delta' : 'full', mode: state.isDelta ? 'delta' : 'full',
compression: state.compression ? 'native' : '', compression:
state.compression === undefined
? undefined
: state.compression
? 'native'
: '',
settings: normalizeSettings({ settings: normalizeSettings({
settings: settings || state.propSettings, settings: settings || state.propSettings,
exportMode: state.exportMode, exportMode: state.exportMode,
@ -313,14 +318,12 @@ export default [
} }
}, },
setVms: (_, vms) => state => ({ ...state, vms }), setVms: (_, vms) => state => ({ ...state, vms }),
updateParams: () => (state, { job, schedules }) => { updateParams: () => (_, { job, schedules }) => {
const remotes = const remotes =
job.remotes !== undefined ? destructPattern(job.remotes) : [] job.remotes !== undefined ? destructPattern(job.remotes) : []
const srs = job.srs !== undefined ? destructPattern(job.srs) : [] const srs = job.srs !== undefined ? destructPattern(job.srs) : []
return { return {
...state,
compression: job.compression === 'native',
name: job.name, name: job.name,
paramsUpdated: true, paramsUpdated: true,
smartMode: job.vms.id === undefined, smartMode: job.vms.id === undefined,
@ -534,11 +537,13 @@ export default [
}, },
}), }),
injectState, injectState,
({ state, effects, remotesById, job }) => { ({ state, effects, remotesById, job = {} }) => {
const { propSettings, settings = propSettings } = state const { propSettings, settings = propSettings } = state
const { concurrency, reportWhen = 'failure', offlineSnapshot, timeout } = const { concurrency, reportWhen = 'failure', offlineSnapshot, timeout } =
settings.get('') || {} settings.get('') || {}
const { compression = job.compression === 'native' } = state
if (state.needUpdateParams) { if (state.needUpdateParams) {
effects.updateParams() effects.updateParams()
} }
@ -597,7 +602,7 @@ export default [
{state.showCompression && ( {state.showCompression && (
<label> <label>
<input <input
checked={state.compression} checked={compression}
name='compression' name='compression'
onChange={effects.setCheckboxValue} onChange={effects.setCheckboxValue}
type='checkbox' type='checkbox'