code cleanup

Author: drivardxrm

IconTwoOption/ControlManifest.Input.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <manifest>
-  <control namespace="DR" constructor="IconTwoOption" version="1.0.5" display-name-key="IconTwoOption" description-key="TwoOption control displaying icons" control-type="standard" preview-image="img/icontwooption-preview.png">
+  <control namespace="DR" constructor="IconTwoOption" version="1.0.6" display-name-key="IconTwoOption" description-key="TwoOption control displaying icons" control-type="standard" preview-image="img/icontwooption-preview.png">
     <!-- property node identifies a specific, configurable piece of data that the control expects from CDS -->
     <property name="twooption" display-name-key="Two Option Field" description-key="Bounded Two-Option field" of-type="TwoOptions" usage="bound" required="true" />
     <property name="lefticon" display-name-key="Left Side Icon" description-key="Left side icon, list here  https://developer.microsoft.com/en-us/fabric#/styles/web/icons#fabric-icons-tool" of-type="SingleLine.Text" usage="input" required="true" default-value="Dislike"/>

IconTwoOption/index.ts

@@ -61,25 +61,18 @@ export class IconTwoOption implements ComponentFramework.StandardControl<IInputs
 	 */
 	public updateView(context: ComponentFramework.Context<IInputs>): void
 	{
-		//Visibility of the main attribute on the form
-		let isVisible = context.mode.isVisible 
-		
+
 		// If the bound attribute is disabled because it is inactive or the user doesn't have access
 		let isReadOnly = context.mode.isControlDisabled;
 
 		let isMasked = false;
 		// When a field has FLS enabled, the security property on the attribute parameter is set
 		if (context.parameters.twooption.security) {
 			isReadOnly = isReadOnly || !context.parameters.twooption.security.editable;
-			isVisible = isVisible && context.parameters.twooption.security.readable;
-			isMasked = isVisible && !context.parameters.twooption.security.readable
+			
+			isMasked = !context.parameters.twooption.security.readable
 		}
 
-		if(!isVisible){
-			return;
-		}
-		
-		
 		//LEFT  TEXT
 		let options:ComponentFramework.PropertyHelper.FieldPropertyMetadata.TwoOptionMetadata | undefined
 				= context.parameters.twooption.attributes;
@@ -97,8 +90,8 @@ export class IconTwoOption implements ComponentFramework.StandardControl<IInputs
 		}
 
 		// Add code to update control view
-		this._selected = context.parameters.twooption.raw; //!== null ? context.parameters.twooption.raw : undefined;
-		//this._selected = context.parameters.twooption.raw;
+		this._selected = context.parameters.twooption.raw; 
+		
 		this._props.selected = this._selected;
 		this._props.lefticon = context.parameters.lefticon.raw || "";
 		this._props.righticon = context.parameters.righticon.raw || "";