Merge pull request scandipwa#4867 from AzizKHAN030/4827_fileResetIssue

Updating the state of productActions component on fieldReset
scandipwa · Jun 20, 2022 · 8f61245 · 8f61245
2 parents d565967 + b2e8473
commit 8f61245
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 6 deletions.
diff --git a/packages/scandipwa/src/component/AddToCart/AddToCart.container.js b/packages/scandipwa/src/component/AddToCart/AddToCart.container.js
@@ -55,6 +55,7 @@ export class AddToCartContainer extends PureComponent {
         addToCart: PropTypes.func,
         fallbackAddToCart: PropTypes.func.isRequired,
         isDisabled: PropTypes.bool,
+        updateSelectedValues: PropTypes.func,
 
         isIconEnabled: PropTypes.bool,
         mix: MixType,
@@ -69,6 +70,7 @@ export class AddToCartContainer extends PureComponent {
         isIconEnabled: true,
         isDisabled: false,
         addToCart: null,
+        updateSelectedValues: null,
         product: {}
     };
 
@@ -95,7 +97,8 @@ export class AddToCartContainer extends PureComponent {
     };
 
     async addProductToCart(e) {
-        const { product, addToCart } = this.props;
+        const { product, addToCart, updateSelectedValues } = this.props;
+        await updateSelectedValues();
 
         if ((!product || Object.keys(product).length === 0) && !addToCart) {
             return;

diff --git a/packages/scandipwa/src/component/Field/Field.container.js b/packages/scandipwa/src/component/Field/Field.container.js
@@ -40,7 +40,8 @@ export class FieldContainer extends PureComponent {
         value: PropTypes.number.isRequired,
         changeValueOnDoubleClick: PropTypes.bool,
         isSortSelect: PropTypes.bool,
-        resetFieldValue: PropTypes.func.isRequired,
+        updateSelectedValues: PropTypes.func,
+        resetFieldValue: PropTypes.func,
         // Validation
         validationRule: ValidationRuleType,
         validateOn: PropTypes.arrayOf(PropTypes.string),
@@ -67,7 +68,9 @@ export class FieldContainer extends PureComponent {
         subLabel: '',
         elemRef: null,
         changeValueOnDoubleClick: false,
-        isSortSelect: false
+        isSortSelect: false,
+        updateSelectedValues: null,
+        resetFieldValue: null
     };
 
     state = {
@@ -121,6 +124,8 @@ export class FieldContainer extends PureComponent {
     }
 
     resetFieldValue(fieldHandler, event) {
+        const { updateSelectedValues } = this.props;
+
         event.preventDefault();
         event.stopPropagation();
 
@@ -130,8 +135,10 @@ export class FieldContainer extends PureComponent {
             isLoading: false
         });
 
+        this.fieldRef.fileData = '';
         this.fieldRef.value = '';
         this.validate();
+        updateSelectedValues();
     }
 
     handleShowLengthError() {

diff --git a/packages/scandipwa/src/component/Product/Product.component.js b/packages/scandipwa/src/component/Product/Product.component.js
@@ -266,7 +266,8 @@ export class Product extends PureComponent {
             addToCart,
             inStock,
             quantity,
-            getActiveProduct
+            getActiveProduct,
+            updateSelectedValues
         } = this.props;
 
         return (
@@ -276,6 +277,7 @@ export class Product extends PureComponent {
               isDisabled={ !inStock }
               isIconEnabled={ false }
               layout={ layout }
+              updateSelectedValues={ updateSelectedValues }
               quantity={ quantity }
               product={ getActiveProduct() }
             />

diff --git a/packages/scandipwa/src/component/Product/Product.container.js b/packages/scandipwa/src/component/Product/Product.container.js
@@ -294,14 +294,21 @@ export class ProductContainer extends PureComponent {
 
         const values = getFieldsData(current, true, [FIELD_TYPE.numberWithControls]);
 
-        values.forEach(({ name, value, type }) => {
+        values.forEach(({
+            field, name, value, type
+        }) => {
             if (type === FIELD_TYPE.select) {
                 selectedOptions.push(value);
             } else if (type === FIELD_TYPE.checkbox || type === FIELD_TYPE.radio) {
                 if (value !== FIELD_RADIO_NONE) {
                     selectedOptions.push(value);
                 }
-            } else if (type !== FIELD_TYPE.numberWithControls) {
+            } else if (type !== FIELD_TYPE.numberWithControls && type !== FIELD_TYPE.file) {
+                enteredOptions.push({
+                    uid: name,
+                    value
+                });
+            } else if (type === FIELD_TYPE.file && field.value) {
                 enteredOptions.push({
                     uid: name,
                     value

diff --git a/.../scandipwa/src/component/ProductCustomizableOption/ProductCustomizableOption.component.js b/.../scandipwa/src/component/ProductCustomizableOption/ProductCustomizableOption.component.js
@@ -174,6 +174,7 @@ export class ProductCustomizableOption extends PureComponent {
                       name: uid,
                       accept: fileExtensions
                   } }
+                  updateSelectedValues={ updateSelectedValues }
                   events={ {
                       onChange: updateSelectedValues
                   } }