From e356c2a5a38a1da18c42f5a9d8e8af5b8b82f9a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Guillermo=20Guti=C3=A9rrez?= <guillegr123@gmail.com>
Date: Fri, 14 Aug 2020 14:48:00 -0600
Subject: [PATCH] [4.3] MSPB-133: Set device defaults on creation only (#264)

---
 submodules/devices/devices.js | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/submodules/devices/devices.js b/submodules/devices/devices.js
index bc65d58..bc5f235 100644
--- a/submodules/devices/devices.js
+++ b/submodules/devices/devices.js
@@ -935,6 +935,7 @@ define(function(require) {
 		 */
 		devicesFormatData: function(data, dataList) {
 			var self = this,
+				isNewDevice = !_.has(data.device, 'id'),
 				keyActionsMod = _.get(
 					self.appFlags.devices.provisionerConfigFlags,
 					['brands', _.get(data.device, 'provision.endpoint_brand'), 'keyFunctions'],
@@ -947,7 +948,14 @@ define(function(require) {
 				isClassifierDisabledByAccount = function isClassifierDisabledByAccount(classifier) {
 					return _.get(data.accountLimits, ['call_restriction', classifier, 'action']) === 'deny';
 				},
-				deviceDefaults = {
+				templateDefaults = {
+					media: {
+						audio: {},
+						encryption: {},
+						video: {}
+					}
+				},
+				deviceBaseDefaults = {
 					call_restriction: {},
 					device_type: 'sip_device',
 					enabled: true,
@@ -1007,6 +1015,7 @@ define(function(require) {
 					smartphone: _.merge({}, sipSettings, callForwardSettings),
 					softphone: _.merge({}, sipSettings)
 				}, data.device.device_type, {}),
+				deviceDefaults = _.merge({}, deviceBaseDefaults, deviceDefaultsForType),
 				deviceOverrides = {
 					provision: _
 						.chain(data.template)
@@ -1032,8 +1041,8 @@ define(function(require) {
 				},
 				deviceData = _.mergeWith(
 					{},
-					deviceDefaults,
-					deviceDefaultsForType,
+					templateDefaults,
+					isNewDevice && deviceDefaults,
 					data.device,
 					function(dest, src) {
 						return _.every([dest, src], _.isArray) ? src : undefined;