From c68aba675f3c2d30c4942cc0b17ec475f964ca7c Mon Sep 17 00:00:00 2001
From: ValerioC <v.castellana@integry.it>
Date: Tue, 26 May 2020 17:38:08 +0200
Subject: [PATCH 1/2] [ FRUDIS ] Inserito controllo risorse ordine = 0
 Ottimizzata gestione liste e observables in lista ordini

---
 .../core/model/CheckableOrdineLavoro.java     | 28 +-------
 .../core/ChooseOrdsLavFromListAdapter.java    | 70 +++++++++----------
 2 files changed, 36 insertions(+), 62 deletions(-)

diff --git a/app/src/main/java/it/integry/integrywmsnative/core/model/CheckableOrdineLavoro.java b/app/src/main/java/it/integry/integrywmsnative/core/model/CheckableOrdineLavoro.java
index 464e2f46..a5ca465c 100644
--- a/app/src/main/java/it/integry/integrywmsnative/core/model/CheckableOrdineLavoro.java
+++ b/app/src/main/java/it/integry/integrywmsnative/core/model/CheckableOrdineLavoro.java
@@ -13,17 +13,11 @@ public class CheckableOrdineLavoro {
 
     private ChooseOrdsLavFromListItemModel item;
     private BindableBoolean checked = new BindableBoolean(false);
-    private BindableBoolean hidden = new BindableBoolean(false);
+
     private String testata;
 
     public CheckableOrdineLavoro(ChooseOrdsLavFromListItemModel item) {
         this.item = item;
-        this.item.getHidden().addOnPropertyChangedCallback(new Observable.OnPropertyChangedCallback() {
-            @Override
-            public void onPropertyChanged(Observable sender, int propertyId) {
-                hidden.set(item.isHidden());
-            }
-        });
     }
 
     public ChooseOrdsLavFromListItemModel getItem() {
@@ -33,13 +27,6 @@ public class CheckableOrdineLavoro {
     public CheckableOrdineLavoro setItem(ChooseOrdsLavFromListItemModel item) {
         this.item = item;
 
-        this.item.getHidden().addOnPropertyChangedCallback(new Observable.OnPropertyChangedCallback() {
-            @Override
-            public void onPropertyChanged(Observable sender, int propertyId) {
-                hidden.set(item.isHidden());
-            }
-        });
-
         return this;
     }
 
@@ -52,19 +39,6 @@ public class CheckableOrdineLavoro {
         return this;
     }
 
-    public BindableBoolean getHidden() {
-        return hidden;
-    }
-
-    public boolean isHidden() {
-        return hidden.get();
-    }
-
-    public CheckableOrdineLavoro setHidden(BindableBoolean hidden) {
-        this.hidden = hidden;
-        return this;
-    }
-
     public BindableFloat getQtaCol() {
         return item.getOrdineLav().getQtaColVersamento();
     }
diff --git a/app/src/main/java/it/integry/integrywmsnative/gest/prod_versamento_materiale/core/ChooseOrdsLavFromListAdapter.java b/app/src/main/java/it/integry/integrywmsnative/gest/prod_versamento_materiale/core/ChooseOrdsLavFromListAdapter.java
index 77e0c8de..d1e9bdfa 100644
--- a/app/src/main/java/it/integry/integrywmsnative/gest/prod_versamento_materiale/core/ChooseOrdsLavFromListAdapter.java
+++ b/app/src/main/java/it/integry/integrywmsnative/gest/prod_versamento_materiale/core/ChooseOrdsLavFromListAdapter.java
@@ -1,6 +1,8 @@
 package it.integry.integrywmsnative.gest.prod_versamento_materiale.core;
 
 import android.content.Context;
+import android.text.Html;
+import android.text.SpannableString;
 import android.view.LayoutInflater;
 import android.view.ViewGroup;
 
@@ -17,6 +19,7 @@ import java.util.HashMap;
 import java.util.List;
 
 import it.integry.integrywmsnative.R;
+import it.integry.integrywmsnative.core.di.BindableBoolean;
 import it.integry.integrywmsnative.core.exception.DateNotRecognizedException;
 import it.integry.integrywmsnative.core.exception.TimeNotRecognizedException;
 import it.integry.integrywmsnative.core.model.CheckableOrdineLavoro;
@@ -29,6 +32,7 @@ import it.integry.integrywmsnative.core.utility.UtilityResources;
 import it.integry.integrywmsnative.core.utility.UtilityString;
 import it.integry.integrywmsnative.databinding.FragmentChooseOrdsLavFromListItemModelBinding;
 import it.integry.integrywmsnative.gest.prod_versamento_materiale.dto.OrdineLavorazioneDTO;
+import it.integry.integrywmsnative.view.dialogs.base.DialogSimpleMessageHelper;
 
 public class ChooseOrdsLavFromListAdapter extends RecyclerView.Adapter<ChooseOrdsLavFromListAdapter.ViewHolder> {
 
@@ -48,40 +52,18 @@ public class ChooseOrdsLavFromListAdapter extends RecyclerView.Adapter<ChooseOrd
             mDataset = Stream.of(myDataset)
                     .withoutNulls()
                     .map(CheckableOrdineLavoro::new).toList();
-
-            for (int i = 0; i < mDataset.size(); i++) {
-                final CheckableOrdineLavoro itemModel = mDataset.get(i);
-                OrdineLavorazioneDTO ordine = itemModel.getItem().getOrdineLav();
-                itemModel.setTestata(UtilityString.formatHtmlStringFromResId(R.string.ord_testata, String.valueOf(ordine.getNumOrd()), UtilityDate.formatDate(ordine.getDateOrd(), UtilityDate.COMMONS_DATE_FORMATS.DMY_HUMAN)).toString());
-                mDatasetPositions.put(itemModel, i);
-
-                itemModel.getHidden().addOnPropertyChangedCallback(new Observable.OnPropertyChangedCallback() {
-                    @Override
-                    public void onPropertyChanged(Observable sender, int propertyId) {
-                        onItemHidden(itemModel);
-                    }
-                });
-                itemModel.getChecked().addOnPropertyChangedCallback(new Observable.OnPropertyChangedCallback() {
-                    @Override
-                    public void onPropertyChanged(Observable sender, int propertyId) {
-                        onItemChecked(itemModel);
-                    }
-                });
-            }
         }
     }
 
-    private void onItemHidden(CheckableOrdineLavoro itemModel) {
-        if (itemModel.isHidden()) {
-            mDataset.remove(itemModel);
-        } else {
-            mDataset.add(mDatasetPositions.get(itemModel), itemModel);
-        }
-        calculateMtbColtShare();
-        notifyDataSetChanged();
-    }
-
     private void onItemChecked(CheckableOrdineLavoro itemModel) {
+        if (itemModel.isChecked() && itemModel.getItem().getOrdineLav().getHrNum() <= 0){
+            DialogSimpleMessageHelper.makeWarningDialog(mContext,
+                    new SpannableString(Html.fromHtml("Nessuna risorsa umana registrata per l'ordine selezionato! <br> Per una corretta suddivisione delle quantità registra le risorse.")),
+                    null, () -> {
+                        itemModel.getChecked().set(false);
+                    }).show();
+            return;
+        }
         if (!itemModel.isChecked()) {
             itemModel.setQtaCol(0);
             itemModel.setNumCnf(0);
@@ -141,7 +123,7 @@ public class ChooseOrdsLavFromListAdapter extends RecyclerView.Adapter<ChooseOrd
         }
     }
 
-    public static class ViewHolder extends RecyclerView.ViewHolder {
+    public class ViewHolder extends RecyclerView.ViewHolder {
         protected FragmentChooseOrdsLavFromListItemModelBinding mViewDataBinding;
 
 
@@ -150,10 +132,21 @@ public class ChooseOrdsLavFromListAdapter extends RecyclerView.Adapter<ChooseOrd
             mViewDataBinding = v;
         }
 
-        public void bind(CheckableOrdineLavoro checkableOrdineLavoro, MtbColr mtbColr) {
+        public void bind(CheckableOrdineLavoro checkableOrdineLavoro, MtbColr mtbColr) throws ParseException, TimeNotRecognizedException, DateNotRecognizedException {
             mViewDataBinding.setCheckableOrdineLav(checkableOrdineLavoro);
             mViewDataBinding.setMtbColr(mtbColr);
-            mViewDataBinding.executePendingBindings();
+
+            OrdineLavorazioneDTO ordine = checkableOrdineLavoro.getItem().getOrdineLav();
+
+            checkableOrdineLavoro.setTestata(UtilityString.formatHtmlStringFromResId(R.string.ord_testata, String.valueOf(ordine.getNumOrd()), UtilityDate.formatDate(ordine.getDateOrd(), UtilityDate.COMMONS_DATE_FORMATS.DMY_HUMAN)).toString());
+
+            checkableOrdineLavoro.getChecked().resetOnPropertyChangedCallback();
+            checkableOrdineLavoro.getChecked().addOnPropertyChangedCallback(new Observable.OnPropertyChangedCallback() {
+                @Override
+                public void onPropertyChanged(Observable sender, int propertyId) {
+                    onItemChecked(checkableOrdineLavoro);
+                }
+            });
 
             if (!SettingsManager.iDB().isFlagForceAllToColli() && (mtbColr.getMtbAart() == null || mtbColr.getMtbAart().isFlagQtaCnfFissa())) {
                 String text = UtilityNumber.decimalToString(checkableOrdineLavoro.getQtaCol().get());
@@ -167,7 +160,6 @@ public class ChooseOrdsLavFromListAdapter extends RecyclerView.Adapter<ChooseOrd
                 mViewDataBinding.qtaTextview.setText(UtilityNumber.decimalToString(checkableOrdineLavoro.getNumCnf().get()) + "\n" + UtilityResources.getString(R.string.unt_mis_col));
             }
 
-            mViewDataBinding.setMtbColr(mtbColr);
             mViewDataBinding.executePendingBindings();
         }
     }
@@ -183,7 +175,15 @@ public class ChooseOrdsLavFromListAdapter extends RecyclerView.Adapter<ChooseOrd
     @Override
     public void onBindViewHolder(ChooseOrdsLavFromListAdapter.ViewHolder holder, int position) {
         CheckableOrdineLavoro item = mDataset.get(position);
-        holder.bind(item, mMtbColr);
+        try {
+            holder.bind(item, mMtbColr);
+        } catch (ParseException e) {
+            e.printStackTrace();
+        } catch (TimeNotRecognizedException e) {
+            e.printStackTrace();
+        } catch (DateNotRecognizedException e) {
+            e.printStackTrace();
+        }
 
     }
 

From b9cbc7759337d9d04dbd8a3f8f4d8305824a821a Mon Sep 17 00:00:00 2001
From: GiuseppeS <g.scorrano@integry.it>
Date: Tue, 26 May 2020 18:46:04 +0200
Subject: [PATCH 2/2] ->-> v1.12.12 (141)

---
 app/build.gradle | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index ab17f67b..c08014f7 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -17,8 +17,8 @@ apply plugin: 'com.google.gms.google-services'
 
 android {
 
-    def appVersionCode = 140
-    def appVersionName = '1.12.11'
+    def appVersionCode = 141
+    def appVersionName = '1.12.12'
 
     signingConfigs {
         release {