From f29b52f9954d1291670142223b5e024580685632 Mon Sep 17 00:00:00 2001
From: Elena <Elena@narvaling.com>
Date: Wed, 29 Jun 2022 00:39:15 +0200
Subject: [PATCH] =?UTF-8?q?Ciclo=20limpieza=20lo=20hace=20completo,=20sect?=
 =?UTF-8?q?oriza=20y=20planifica.=20Falta=20probar=20instalaci=C3=B3n.=20R?=
 =?UTF-8?q?ecogida=20a=20medias=20de=20probar.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Config.daml                                 |   2 +-
 Helper/HelperGdb.cs                         |  43 +++++--
 Images/config2_olv - copia.png              | Bin 0 -> 1789 bytes
 Images/config2_olv.png                      | Bin 0 -> 1906 bytes
 Model/Limpieza.cs                           |   9 +-
 Model/Recogida.cs                           |  21 +++
 OliviaAddInPro.csproj                       |   5 +-
 Services/EjecServ.cs                        | 136 +++++++++++++-------
 Services/FinProcServ.cs                     |  35 ++---
 Services/RecogidaServ.cs                    |  25 +---
 View/PaneEjecutar.xaml.cs                   |   2 +-
 ViewModel/Recogida/PaneRecogidaViewModel.cs |   3 +
 12 files changed, 183 insertions(+), 98 deletions(-)
 create mode 100644 Images/config2_olv - copia.png
 create mode 100644 Images/config2_olv.png

diff --git a/Config.daml b/Config.daml
index 1e55652..882f6e4 100644
--- a/Config.daml
+++ b/Config.daml
@@ -38,7 +38,7 @@
           <tooltip heading="OLIVIA - Recogida de Residuos">
 			  Add-In de ArcGIS Pro para la herramienta de optimización de la limpieza viaria, OLIVIA<disabledText /></tooltip>
         </button>
-        <button id="OliviaAddInPro_ButtonConfig" caption="Configuración" className="ButtonConfig" loadOnClick="true" largeImage="Images/config2.png">
+        <button id="OliviaAddInPro_ButtonConfig" caption="Configuración" className="ButtonConfig" loadOnClick="true" largeImage="Images/config2_olv.png">
           <tooltip heading="OLIVIA - Configuración">
 			  Add-In de ArcGIS Pro para la herramienta de optimización de la limpieza viaria, OLIVIA<disabledText /></tooltip>
         </button>        
diff --git a/Helper/HelperGdb.cs b/Helper/HelperGdb.cs
index 4368a00..665824f 100644
--- a/Helper/HelperGdb.cs
+++ b/Helper/HelperGdb.cs
@@ -739,9 +739,10 @@ namespace OliviaAddInPro.Helper
             if (!fields.Contains(field))
             {
                 HelperGdb.OutStr = "No se encuentra el campo " + field;
+                Free(fc);
                 return false;
             }
-
+            Free(fc);
             return true;
         }
         /*
@@ -757,13 +758,14 @@ namespace OliviaAddInPro.Helper
             }
 
             ObservableCollection<string> fields = GetFieldsSync(fc);
+            bool encuentra = true;
             if (!fields.Contains(field))
             {
                 HelperGdb.OutStr = "No se encuentra el campo " + field;
-                return false;
+                encuentra= false;
             }
-
-            return true;
+            Free(fc);
+            return encuentra;
         }
         /*
          * Lee la capa que se ha seleccionzdo de recogida y se comprueba que contiene los campos necesarios
@@ -1942,12 +1944,27 @@ namespace OliviaAddInPro.Helper
             if (brwsFilt != null)
                 dlg.BrowseFilter = brwsFilt;
 
-            bool? ok = dlg.ShowDialog();
+            try
+            {
+                bool? ok = dlg.ShowDialog();
 
-            if ((ok ?? true) && dlg.FilePath.Length > 0)
-                return dlg.FilePath;
-            else
+                if ((ok ?? true) && dlg.FilePath.Length > 0)
+                    return dlg.FilePath;
+                else
+                    return string.Empty;
+            }
+            catch(Exception ex)
+            {
                 return string.Empty;
+            }
+        }
+
+        /**
+         * Saca diálogo para elegir directorio
+         */
+        public static string FolderBrowseDlg(string title_, string initloc_)
+        {
+            return null;
         }
 
         /**
@@ -2438,12 +2455,17 @@ namespace OliviaAddInPro.Helper
          */
         public static bool AddFieldsSync(string fcname, FieldToAdd[] fields)
         {
-            bool res = false;
+            bool res = false;            
 
             // set up the arguments
             string fields_str = "";
+            int camps = 0;
             for(int i=0;i<fields.Length;i++)
             {
+                if (CheckFieldSync(fcname, fields[i].Name))
+                    continue;
+                camps++;
+
                 fields_str += fields[i].Name + " " + fields[i].Tipo + " ";
                 if (!string.IsNullOrEmpty(fields[i].Alias))
                     fields_str += fields[i].Alias;
@@ -2460,6 +2482,9 @@ namespace OliviaAddInPro.Helper
                 if (i < fields.Length - 1)
                     fields_str += ";";
             }
+            if (camps == 0)
+                return true;//no tiene que añadir ningún campo porque los tiene
+
             //var f1 = "field1 long field1 # #;field2 Text # 255 myDefaultValue"; //los # son opciones sin rellenar el orden es: Nombre Tipo Alias Longitud Valor Defecto
 
             var args = Geoprocessing.MakeValueArray(fcname, fields_str);
diff --git a/Images/config2_olv - copia.png b/Images/config2_olv - copia.png
new file mode 100644
index 0000000000000000000000000000000000000000..cec6441a42b776dff3e9b0b61e4ed465d58762e7
GIT binary patch
literal 1789
zcmaJ?X;2eq7+&r}tQFHzFS;a9u$XKTLLh5W2{|BWKq2L@&>E7Jq#@b3StLLe#frtY
zsG#*2<?;ZuP((xoid?p%qS%TZK|qQ^!3yfAgMdmm2-qK`JG=XR-#gFqyywnltO*X7
zYrn`I0D!rYKw&7cf~LpTns^(Mlb;dGLR=JyhoKwrL^*~4d?gx-fD)}d4hco%%A}2#
z5iS5&xT<B5c%*bStU$Fixd}rvYV`yg0JtlSdbvV_;9x8gr`GYvk7_QGLA8=cj_{V!
zrFsFPQU@kuNLX^POp&ZnaFpbgpMYE=ObBQZTn-ww@j3%+<dG+JVPbDGLu7Cgf@^r>
zH$g>8*MI^PLqKnu7ga%LFhDk&#$b7~*^K2Nlg?m5bmC@H87$bF4Ksbfsf$c_!;~A~
zP@#Cr7jfc|RXDDPAt*61k(TI1L$NrB!QpUB8cZgYKu`@yI$Um~>I{@=1tDTkU}`<C
zMs=V`Q67sX;5;%B>6;X^`WacBVX90-!62ht4>4$TQ%ci7sr3J$TI~$lfQKS)<NZ%z
zgDgppK%s~MO~4dH<2F!CuJo_~L*zJ$$xt+Yx{7O5D2^Ibs2&uAu|W^1T%p#P49g}N
zQYkFa8F0Byfk=crGND0JtCg^b@6TX!yxHCiA&0>bF@4!$F`q8rdolPTk%+^Z<_b|o
zf)>%?(_H0SuFtGo(=cfDgl8dwsW%}?F@|cv$)sWR?6I(B)tllfXOG2uRxU&Y1DRU;
zuU1c+i1{&XX9kxz%#1#wBjz0=20OF%;RnR`f=YyZnX#ieoUd8HC*{OAu3JT2`(=H{
z4cnhHSjA3|e=na)lJMUR*#LF7J#%V)okmI_Nw^t#@$aPdxS7pA;gRvfx(6pD-<9kL
zI?t=v)iBmQR=K*;E4yo`!6y6K@WbmP*HZ4LJZ1Ga4R;=u)Vo<x$OlUsRV^z$STBDp
zP6EDfa+^n?n2FD>ZyTIgzNg6f5C8<&`no4s@tvDIEb=ayA3h2ER38|)Ndu_bw~v(T
zFNoxZ_`?%y*HJcP7U22Z{6KsjzbD~!Oia1h|IC)g?Q!3hj?kkeK2>zf<IfJqZggmf
zit3b9obIHM`Zl-6x(rifqX+%*%>0x&MHZL8y0Vn{Y5L&f?EIxp+^xs1b=EQ2XJ5q9
zJP+-z8Gf|Pa-CrA!K|0!e)k()wn1bjP#a|KX`XS!m6voiYv(*m`(uId8CF%)du65m
zoxcz7H=_r4_u4J~s}Qr%ZFy0&Z?v{Txd}chC|-5x%6&`t^v&T;D8$opfOc-cE~UXe
zBv)i@WAn5yKrF0gNRHU)N;pTcjvmaL8s__ulv`-!`f*9e=@uo{c>Pu6@?Q#mQ$bX@
zw5$T`%ia|Iryzgl*zJKGns|j~_vWHkt47eF%3JYlYQ#4+_F3;&q+4W=_O5qywB1ve
zbgx=;&W#LHEf)A`pzxNG{$E>M9J9m1@7^d@rlzL09^;TmODrQC0$P#uEfdNW8c&<n
z34F_$KN<}ueXHB|a<>M#iwdo+t#9|Wxm7+Hbh~qB-2!CwqOU)TC@CDb+#S|Agw@p4
zT>boF(pB+CrR8<^nhM%Z`z}jXnghd2LSyFW&_`t*z0AeV7b8!`jFa<IEL*E@W}XlK
zTl4(>p(kqo+Ix~_i}}FgCAYMPiXH^_9nXr++C$$aY<4n}!AG-Bb=O}{cwKRSkt~rC
zwlVug^0F=qW7nz^_j%SiK|#Ep@ajf~`Mx^}N>!1Leq|o6ek;!JcYbH*!u|{9(d`G#
z?T+N`Umds9@vxuXs<0l+NSyPieQa!`RBTm0_rth!$xx1-Qtek#_Ef?4?TC!umkW)X
zCHlD*RHWF&fXgc{j9Oqd+Aj`7)IA>Tcj_E;ap3PQf5O@ZXw08?lvH_ky>z}Cb;><P
zDmh>S<hcwOPo+i2nQ_<la^*g59x`oeYVNC$anE|7)=O2jD<{)v`XNa~!NOAi^=bbA
D&Vsjv

literal 0
HcmV?d00001

diff --git a/Images/config2_olv.png b/Images/config2_olv.png
new file mode 100644
index 0000000000000000000000000000000000000000..01310327c07b183e580c76480e1619e6c8641d32
GIT binary patch
literal 1906
zcmV-&2aWiNP)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F800009a7bBm000XU
z000XU0RWnu7ytkOAY({UO#lFTB>(_`g8%^e{{R4h=l}px2mk>USO5SzmjD14Z`WEM
zkN^M%_(?=TRCwCNR%=XK=NW#^&9~UZ#>Rj#m_tlR1SF<yB1EJV=xFjIok+H+yF@EY
zQ#3_|;7wYzNmI(CEnC&e2zOnSY=TYcswQdhS`=kz7Lzr~ts_9ml{6uMfe=VIHrV)b
zIA=e4u#Ds5knYF6U-tRF?|q){eV_M2;5k-Pv(o{f8h}2<b`O9D?snIM=dnqw0JIRm
z0RZ(fzyQDj=$C%Cy4zhn|1*K6W~W{PZjm|C1K_U!JnnWEhp)MaG~iwU4w=_By4ziz
z6?m@!z#CH7me$V1yR^*R12W(M(D^KInFl<O!21)=k}@_NY3w^z;MtC(b3FhmlEG7k
zKvH^N8si)QRxa+6XonL%wYu9~Z^;>>6E4*Qu%!{*($WKP0ce!juTBnGx(p+mnw|C1
zi(M-K2LR`uS8dM0?zRS*{lcXG`_c%sj|9*qli!^H3;<XFPDw<KdtSBmNCN8^>~3q3
zNzcU9AbB`0(57Z*holpZXliy2r(oUVZg<rrTORCgE0nyINVu_lM@!tzNB+`TfAhwT
zE}rL~N?oZ~btMS2m6TqWe0XOX0VbE)j=}CW2LOEV;R(G~t38lwv82*Xmn2R{f~_`E
z%IU_0T&<B)c=?VN4nR!`65i3iGxaZ2Z1e(f<mczZY&JhjOFP}|E{*_z^_v_pXPHsC
zX)~^TaUE0BGf*p)0Dwh707+3>cD?e-7Hfg+?YZDQYAhj#q=R@UNLmT&i{Hf5>;iiG
zzDoCTpN|_E8R3#bysRKCD?}tgV3kURXf%p&B#KBh27&WqSgrZ_NJ+`>H?J+OwzDjb
zd^X;n943K}z}NdRvJGmySm6MG{yeLdb+@}%02PwwSa-XNO)N-p4`i9Lu(hfR{=hs=
zd3!)J3?PbF9gE@S#toOE`<g2c2g7~~$JL&_bn)7;e|T`=eRtRF$UyzFN6<=GU)aGZ
zGqNgFD%CSv^D+;JA|Q$)Xp(?h!9YtAAdH#cQ`XhkqwCk;gigIWF#YJYfIom={dE6P
z##~yTVJShfCq}P+gu7=C=~vPX$v85A7|+A7Is-XIBW9)_Av8CK@u{gjkLLp>kRr^Y
zFv1hm(U-Su^^W*Pf3WvAzXqSV9%@5M)q9t+<Cz3RfPx5?B+0KrQep>NU<FSTFh;`P
zR$u*m0^_47<NOS%UgZH$Z7(dslc_2E_SpL{uB(AkXZl(sNC433_2j~0^oO$n|J%Z%
zkW14vOeP~J5TG-(g-W|k4Ka+9LdZg?nKFtbz%UH<?0Ow{u6~YF|Naand!<Lp?dKvv
z0suo1F>{u&_4fUt!TY1b+X;}XBH;*PTmZ9L%`dp}H3nJ;<AQ<41%wa{hvNpmwe=hL
zq~}cJ^7WxZDT-Pmi9%Tdlqi;O-&(tIEnD=rq2aq<h8BY53&9}x$O0e;)d&SZEd)`?
z^H?8<fKn*q26D1<`maq-+_}XEPfI1aQ-(y>V0W87C4qS{_6J6<JGh~EE%GecT!v0N
zX=hpd&h=hLU8%!J-?@zlKZl)<!^ogC%S!W~55K#Y(kaU(o=o;Gb<2bSHF=QnCX*lm
zU}kFS^7Z?-@)k*cSJmdqH_A#%&9yIY^8#qdHDyCFItKVBA%KU1rV{%?&Dg})r41_0
zxqKe`6UudC3O&S0JgtHG=sTA$o}cZfX8*X_p1)72P@Y&CEJy{7k`WH(kIv%8z3Z3{
z1>;@t_{qcl`Sv`k%~nusGv=uhdPaYmfyFV{-PW;;07;U8siCo}1HLbF2%VCqX;lHk
zFwkf;D7LSIbN61nX?h9k=2c)6DgbCa`r*4C6{FlpZpb~aWpiwyS^q>*<MIG#l_Alv
zM1lkWwN42Q$ZHRK2QfY~0;55XZFRNSR#%JM+#E0rgW|QNSW~bL06LE!f3H)iR_*6X
zG`%{9rAXxu_T3!1HISlvz6HROhC~Cs)DaXK<tYot44*l5k~P#(h+YZ7=NrdMWt%W5
z-H64apM2^)c4QT+KR2BjtD!9vO2V|TF#6~xp=d;q0cw=GVX9itK><KkwjLsZ$nMvF
zY`AyrMs?rVe=szE8xI#J5nYIYj*;*^yf=CA?mshyT;gZJ%rMI^K+&QuV4PKaCma#I
zf=J@xh09MN{ORwf!+XEK{R)8Z0#GFnIPqpA3IeQFTj!5<zv-{cs)BfX5&fTD!oLPi
z;)^F|F~IfY&Z!==5P~qx(LURZ_K;$1v1(Bex<Qd?>%An(R7I+h0Dw}*8i=9@ilXq#
z{RbV=_*6K2|1Qp7`8Z%H(v6EzoC^n|b6G}1pGu)V&5J@n<O!Iv%?O8Rgck(7bI_HR
sh%`yNq!e|Q?`ZK5qSOihU+cdE0QjcOC{p#G0RR9107*qoM6N<$g8IgP9smFU

literal 0
HcmV?d00001

diff --git a/Model/Limpieza.cs b/Model/Limpieza.cs
index f580ad4..2e704e2 100644
--- a/Model/Limpieza.cs
+++ b/Model/Limpieza.cs
@@ -66,9 +66,12 @@ namespace OliviaAddInPro.Model
             /////
             ///DEBUG
             ///
-            /*Respuesta<TiposEjecucion> res1 = new Respuesta<TiposEjecucion> { Value = TiposEjecucion.FinEjecOk };
-            HelperGdb.OpenLayer(@"C:\Users\Elena\Documents\ArcGIS\Projects\MyProject3\MyProject3.gdb\Barrido_man\BarMan_AceM_AceNM_20220529_134336", "SECTOR", true);
-            return res1;*/
+            //Respuesta<TiposEjecucion> res1 = new Respuesta<TiposEjecucion> { Value = TiposEjecucion.FinEjecOk };
+            //HelperGdb.OpenLayer(@"C:\Users\Elena\Documents\ArcGIS\Projects\MyProject3\MyProject3.gdb\Barrido_man\BarMan_AceM_AceNM_20220529_134336", "SECTOR", true);            
+            //Serv.ActualizaSecto(@"D:\Proyectos\Olivia\Instal2.0\data\data_T00_A0405_20220628_214902.shp", @"C:\\Users\\Elena\\Documents\\ArcGIS\\Projects\\MyProject3\\MyProject3.gdb\\Barrido_man\\BarMan_AceM_AceNM_20220628_214613");
+            //return res1; 
+
+            //////////////////
             /////////////////////////////
 
             Respuesta<bool> res = new Respuesta<bool> { Value=false};
diff --git a/Model/Recogida.cs b/Model/Recogida.cs
index 08e1d7c..bfee6f4 100644
--- a/Model/Recogida.cs
+++ b/Model/Recogida.cs
@@ -171,5 +171,26 @@ namespace OliviaAddInPro.Model
             RecogidaDef.campos_def.cons_kgrec };
             return camps;
         }
+
+        /**
+         * Rellena el array de filtros en base a la config
+         */
+        public void RellenaFiltrosReco()
+        {
+            //se rellena el array de filtros            
+            RecogidaDef.filtro_str = new string[RecogidaDef.tipos_fracc_str.Length + RecogidaDef.tipos_carg_str.Length];
+            for (int i = 0; i < RecogidaDef.tipos_fracc_str.Length; i++)
+            {
+                //se comprueba que tipo de fracción se ha seleccionado. Si se ha seleccionado una fracción sin nombre (el campo en la capa es NULL, y en el comboBox de fracciiones aparece como "-") hay que modificar el filtro
+                if (RecogidaDef.tipos_fracc_str[i] == "-")
+                    RecogidaDef.filtro_str[i] = RecogidaDef.campos_def.cons_fracc + " IS NULL";
+                else
+                    RecogidaDef.filtro_str[i] = RecogidaDef.campos_def.cons_fracc + "= '" + RecogidaDef.tipos_fracc_str[i] + "'";
+            }
+            for (int i = RecogidaDef.tipos_fracc_str.Length; i < RecogidaDef.filtro_str.Length; i++)
+            {
+                RecogidaDef.filtro_str[i] = RecogidaDef.campos_def.cons_nomrec + "= '" + RecogidaDef.tipos_carg_str[i - RecogidaDef.tipos_fracc_str.Length] + "'";
+            }
+        }
     }
 }
diff --git a/OliviaAddInPro.csproj b/OliviaAddInPro.csproj
index afe5a47..e8f1d77 100644
--- a/OliviaAddInPro.csproj
+++ b/OliviaAddInPro.csproj
@@ -25,7 +25,7 @@
     <OutputPath>bin\Debug\</OutputPath>
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
-    <WarningLevel>4</WarningLevel>
+    <WarningLevel>0</WarningLevel>
     <StartAction>Program</StartAction>
     <StartProgram>$(ArcGISFolder)\bin\ArcGISPro.exe</StartProgram>
     <PlatformTarget>x64</PlatformTarget>
@@ -357,6 +357,9 @@
   <ItemGroup>
     <AddInContent Include="Images\carrito2.png" />
   </ItemGroup>
+  <ItemGroup>
+    <AddInContent Include="Images\config2_olv.png" />
+  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!--
   PackageAction can be:
diff --git a/Services/EjecServ.cs b/Services/EjecServ.cs
index 2591a3a..f1ca1af 100644
--- a/Services/EjecServ.cs
+++ b/Services/EjecServ.cs
@@ -56,7 +56,7 @@ namespace OliviaAddInPro.Services
         {
             ErrStr = string.Empty;
             try
-            {
+            {              
                 com.ProgrSrc.Init("Exportando datos");
                 //Comprueba que tiene las columnas necesarias para planificar
                 if ((modo == ModosEjec.Planifica) && !CompruebaPlanif())
@@ -298,53 +298,103 @@ namespace OliviaAddInPro.Services
             if (!HelperGdb.AddFieldsSync(path_shp, fields))
                 return false;
             //vuelve a cerrar la capa
-            string capa_principal = System.IO.Path.GetFileNameWithoutExtension(OliviaGlob.Paths.PathData);
+            string capa_principal = System.IO.Path.GetFileNameWithoutExtension(path_shp);
             HelperGdb.CloseLayer(capa_principal);
-            /////////////////////////////////////////////////////
-            RowCursor rc_shp = null;
-            RowCursor rc_secto = null;
-            Row rowshp = null;
-            Row rowsecto = null;
-            FeatureClass fc_secto = null;
-            FeatureClass fc_shp = null;
-            bool bien = false;
-            try
+            /////////////////////////////////////////////////////       
+                        
+            //ahora rellena las columnas
+            using (FeatureClass fc_secto = HelperGdb.GetFtClassSync(path_secto))
             {
-                //ahora rellena las columnas
-                fc_secto = HelperGdb.GetFtClassSync(path_secto);
-                fc_shp = HelperGdb.GetFtClassSync(path_shp);
-                if (fc_secto == null || fc_shp == null)
-                    return false;
+                using (FeatureClass fc_shp = HelperGdb.GetFtClassSync(path_shp))
+                {
+                    if (fc_secto == null || fc_shp == null)
+                        return false;
 
-                var modifyOp = new ArcGIS.Desktop.Editing.EditOperation();
-                modifyOp.Name = "Actualiza SECTOR y SECUENCIA";
-                //modifyOp.Callback((context) =>
-                //{ 
-                    rc_shp = fc_shp.Search();
-                    rc_secto = fc_secto.Search();
-
-                    if (rc_shp == null || rc_secto == null)
+                    try
+                    {
+                        using (RowCursor rc_shp = fc_shp.Search())
+                        {
+                            using (RowCursor rc_secto = fc_secto.Search())
+                            {
+                                if (rc_shp == null || rc_secto == null)
+                                    return false;
+
+                                var modifyOp = new ArcGIS.Desktop.Editing.EditOperation();
+                                modifyOp.Name = "Actualiza SECTOR y SECUENCIA";
+                                bool ex = false;
+                                //modifyOp.Callback((context) =>
+                                //{
+                                    while (rc_shp.MoveNext() && rc_secto.MoveNext())
+                                    {
+                                        using (Row rowsecto = rc_secto.Current)
+                                        {
+                                            using (Row rowshp = rc_shp.Current)
+                                            {
+                                                //context.Invalidate(rowshp);
+                                                // modify and execute
+                                                modifyOp.Modify(rowshp, LimpiezaDef.Campos.consulta_sector, rowsecto[LimpiezaDef.Campos.consulta_sector]);
+                                                modifyOp.Modify(rowshp, LimpiezaDef.Campos.consulta_secuen, rowsecto[LimpiezaDef.Campos.consulta_secuen]);
+                                                rowshp.Store();
+                                                //context.Invalidate(rowshp);
+                                            }
+                                        }
+                                    }
+                                //}, fc_shp);
+                                ex = modifyOp.Execute();
+                                ArcGIS.Desktop.Core.Project.Current.SaveEditsAsync();
+                                if (ex && modifyOp.IsDone)
+                                    return true;
+                                else
+                                    return false;
+                            }
+                        }
+                    }
+                    catch
                     {
-                        HelperGdb.Free(rc_shp);
-                        HelperGdb.Free(rc_secto);
-                        HelperGdb.Free(fc_secto);
-                        HelperGdb.Free(fc_shp);
                         return false;
                     }
 
-                    while (rc_shp.MoveNext() && rc_secto.MoveNext())
+                }
+            }
+                    
+                
+                /*
+                var modifyOp = new ArcGIS.Desktop.Editing.EditOperation();
+                modifyOp.Name = "Actualiza SECTOR y SECUENCIA";
+                bool ex=false;
+                modifyOp.Callback((context) =>
+                { 
+                using (RowCursor rc_shp = fc_shp.Search())
+                {
+                    using (RowCursor rc_secto = fc_secto.Search())
                     {
-                        rowsecto = rc_secto.Current;
-                        rowshp = rc_shp.Current;
-                        // modify and execute
-                        modifyOp.Modify(rowshp, LimpiezaDef.Campos.consulta_sector,rowsecto[LimpiezaDef.Campos.consulta_sector]);
-                        modifyOp.Modify(rowshp, LimpiezaDef.Campos.consulta_secuen, rowsecto[LimpiezaDef.Campos.consulta_secuen]);
-                    }
-                    bien = true;
-                //}, fc_shp.GetFeatureDataset());
 
-                modifyOp.Execute();
-                if (bien)
+                        if (rc_shp != null && rc_secto != null)
+                        {
+                            while (rc_shp.MoveNext() && rc_secto.MoveNext())
+                            {
+                                using (Row rowsecto = rc_secto.Current)
+                                {
+                                    using (Row rowshp = rc_shp.Current)
+                                    {
+                                        context.Invalidate(rowshp);
+                                        // modify and execute
+                                        modifyOp.Modify(rowshp, LimpiezaDef.Campos.consulta_sector, rowsecto[LimpiezaDef.Campos.consulta_sector]);
+                                        modifyOp.Modify(rowshp, LimpiezaDef.Campos.consulta_secuen, rowsecto[LimpiezaDef.Campos.consulta_secuen]);
+                                        rowshp.Store();
+                                        context.Invalidate(rowshp);
+                                     }
+                                }
+                            }
+                            bien = true;
+                            rc_secto.Dispose();
+                            rc_shp.Dispose();                          
+                        }
+                    }
+                }
+                }, fc_shp);
+                ex = modifyOp.Execute();
+                if (bien && ex && modifyOp.IsDone)
                     return true;
                 else
                     return false;
@@ -355,15 +405,13 @@ namespace OliviaAddInPro.Services
             }
             finally
             {
-                HelperGdb.Free(rowsecto);
-                HelperGdb.Free(rowshp);
-                HelperGdb.Free(rc_shp);
-                HelperGdb.Free(rc_secto);
+                fc_secto.Dispose();
+                fc_shp.Dispose();
                 HelperGdb.Free(fc_secto);
                 HelperGdb.Free(fc_shp);
 
             }
-            return false;
+            return false;*/
         }
 
         /**
diff --git a/Services/FinProcServ.cs b/Services/FinProcServ.cs
index a5ce1f7..4984c6b 100644
--- a/Services/FinProcServ.cs
+++ b/Services/FinProcServ.cs
@@ -120,19 +120,15 @@ namespace OliviaAddInPro.Services
                     }
                     else
                     {
+                        //actualiza la capa de la que tiene que leer ahora para planificar 
+                        inst.CapaPlanif = resp2.Value;
                         if (OliviaGlob.HasFlagTipEjec(TiposEjecucion.EjecPlanif)) //Ha terminado bien la planificación
                         {
                             //guarda csv
                             GuardaCsv(inst);
                         }
-                        msg = Resource1.String_exito;
-                        //actualiza los flags
-                        OliviaGlob.RemoveFlagTipEjec(TiposEjecucion.EjecPlanif); 
-                        OliviaGlob.RemoveFlagTipEjec(TiposEjecucion.EjecSecto);
-                        //pone modo config2
-                        OliviaGlob.AddFlagTipEjec(TiposEjecucion.Config2);
-                        //actualiza la capa de la que tiene que leer ahora para planificar 
-                        inst.CapaPlanif = resp2.Value;
+                        msg = Resource1.String_exito;                       
+                        
                     }
                 }
                 catch (Exception ex)
@@ -141,7 +137,7 @@ namespace OliviaAddInPro.Services
                     mal = true;
                 }
             }
-            if (mal)
+            /*if (mal)
             {
                 //reinicia flags
                 if (OliviaGlob.IsLimp())
@@ -149,7 +145,13 @@ namespace OliviaAddInPro.Services
                 else if (OliviaGlob.IsReco())
                     OliviaGlob.SetFlagTipEjec(TiposEjecucion.Reco);
 
-            }
+            }*/
+            //actualiza los flags
+            OliviaGlob.RemoveFlagTipEjec(TiposEjecucion.EjecPlanif);
+            OliviaGlob.RemoveFlagTipEjec(TiposEjecucion.EjecSecto);
+            //pone modo config2
+            OliviaGlob.AddFlagTipEjec(TiposEjecucion.Config2);
+            ///////////////////////////////////////////////////
             HelperGlobal.ponMsg(msg);
             OliviaGlob.progrDialog.Hide();
             //muestra la ventana
@@ -214,16 +216,15 @@ namespace OliviaAddInPro.Services
             nameDokL = Directory.GetFiles(DirData,
                 System.IO.Path.GetFileNameWithoutExtension(OliviaGlob.Paths.PathData) + inst.ServCom.NAME_CSV + ".*");
 
-            string Title = "Guardar Secuencia de la Planificación";
+            string Title = "Directorio para guardar Secuencia de la Planificación";
             string Filter = "Secuencia en formato CSV (*.csv)|*.csv";
             inst.decode_gdb(System.IO.Path.GetFileNameWithoutExtension(OliviaGlob.Paths.PathData),
                     out auxi, out nombre);
-            string InitialDirectory = System.IO.Path.Combine(DirData, nombre);
-            
 
-            string FileName = HelperGdb.SaveFileDlg(Title, InitialDirectory, inst.ServCom.EXT_CSV, Filter);
+            //string Path = HelperGdb.SaveFileDlg(Title, DirData, inst.ServCom.EXT_CSV, Filter);            
+            string Path = HelperGdb.SaveFileDlg(Title, DirData, null, ArcGIS.Desktop.Catalog.ItemFilters.folders);
 
-            if (string.IsNullOrEmpty(FileName) ||  FileName.Length == 0)
+            if (string.IsNullOrEmpty(Path) || Path.Length == 0)
             {
                 bool ok = HelperGlobal.ponMsg("Se va a perder el archivo que contiene la secuencia en planificación, ¿está seguro?", 
                     MessageBoxImage.Question,"OLIVIA",MessageBoxButton.YesNo);
@@ -232,6 +233,8 @@ namespace OliviaAddInPro.Services
             }
             else
             {
+                var FileName = System.IO.Path.Combine(Path, nombre);
+                FileName += inst.ServCom.EXT_CSV;
                 if (File.Exists(FileName))
                     File.Delete(FileName);
                 File.Move(nameDokL[0], FileName);
@@ -253,7 +256,7 @@ namespace OliviaAddInPro.Services
                 //repite por si se ha equivocado hasta que elige la gdb
                 GdbFileName = HelperGdb.OpenFileDialog(HelperGdb.TiposOpenFileDlg.OpenGdb,"", "Seleccionar GDB a la que importar los resultados");
                 if(string.IsNullOrEmpty(GdbFileName))
-                    sal= HelperGlobal.ponMsg("¿Desea cancelar el proceso de imporación?",
+                    sal= HelperGlobal.ponMsg("¿Desea cancelar el proceso de importación?",
                      MessageBoxImage.Question, "OLIVIA", MessageBoxButton.YesNo);
             } while (!sal);
 
diff --git a/Services/RecogidaServ.cs b/Services/RecogidaServ.cs
index 0789b86..1c9154b 100644
--- a/Services/RecogidaServ.cs
+++ b/Services/RecogidaServ.cs
@@ -18,7 +18,6 @@ namespace OliviaAddInPro.Services
         public RecogidaServ(Recogida _reco)
         {
             reco = _reco;
-            RellenaFiltrosReco();
         }
 
         /*
@@ -95,7 +94,7 @@ namespace OliviaAddInPro.Services
 
             //cierra las capas que se han abierto durante la exportación
             CierraCapas();
-
+            res.Value = true;
             return res;
 
         }
@@ -145,27 +144,7 @@ namespace OliviaAddInPro.Services
                     consulta = "";
             }
             return consulta;
-        }
+        }       
         
-        /**
-         * Rellena el array de filtros en base a la config
-         */
-         public void RellenaFiltrosReco()
-        {
-            //se rellena el array de filtros            
-            RecogidaDef.filtro_str = new string[RecogidaDef.tipos_fracc_str.Length + RecogidaDef.tipos_carg_str.Length];
-            for (int i = 0; i < RecogidaDef.tipos_fracc_str.Length; i++)
-            {
-                //se comprueba que tipo de fracción se ha seleccionado. Si se ha seleccionado una fracción sin nombre (el campo en la capa es NULL, y en el comboBox de fracciiones aparece como "-") hay que modificar el filtro
-                if (RecogidaDef.tipos_fracc_str[i] == "-")
-                    RecogidaDef.filtro_str[i] = RecogidaDef.campos_def.cons_fracc + " IS NULL";
-                else
-                    RecogidaDef.filtro_str[i] = RecogidaDef.campos_def.cons_fracc + "= '" + RecogidaDef.tipos_fracc_str[i] + "'";
-            }
-            for (int i = RecogidaDef.tipos_fracc_str.Length; i < RecogidaDef.filtro_str.Length; i++)
-            {
-                RecogidaDef.filtro_str[i] = RecogidaDef.campos_def.cons_nomrec + "= '" + RecogidaDef.tipos_carg_str[i - RecogidaDef.tipos_fracc_str.Length] + "'";
-            }
-        }
     }
 }
diff --git a/View/PaneEjecutar.xaml.cs b/View/PaneEjecutar.xaml.cs
index 5c4113c..572bda1 100644
--- a/View/PaneEjecutar.xaml.cs
+++ b/View/PaneEjecutar.xaml.cs
@@ -46,7 +46,7 @@ namespace OliviaAddInPro
             }
             else if (DataContext is PaneRecogidaViewModel modrec)
             {
-                modrec.Ejecuta(OliviaAddInPro.Services.ModosEjec.Planifica);
+                modrec.Ejecuta(OliviaAddInPro.Services.ModosEjec.SoloPlanifica);
             }
 
         }
diff --git a/ViewModel/Recogida/PaneRecogidaViewModel.cs b/ViewModel/Recogida/PaneRecogidaViewModel.cs
index ccf9e63..741ef7d 100644
--- a/ViewModel/Recogida/PaneRecogidaViewModel.cs
+++ b/ViewModel/Recogida/PaneRecogidaViewModel.cs
@@ -94,6 +94,9 @@ namespace OliviaAddInPro
             err_str = "";
             try
             {
+                //Rellena el array de filtros
+                reco.RellenaFiltrosReco();
+
                 ////////////////////////////////////////////////////////////
                 ///Lee el panel 1
                 if (!_subPanel1ViewModel.CapaAbierta || string.IsNullOrEmpty(_subPanel1ViewModel.CapaElems))