search

Lucas128 / libertya

Automatically exported from code.google.com/p/libertya
0 stars 0 forks source link

Importación de Cuentas Contables - No se actualizan cuentas por default #15

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Importar archivo CSV con un plan de cuentas diferente al que viene con la 
distribución de Libertya
2. Procesar la importación, seleccionando "Actualizar cuentas por defecto" y 
"Crear nueva combinación"

What is the expected output? What do you see instead?
Ir a Esquema Contable -> Predeterminados, allí las cuentas por default 
deberían mostrar las cuentas recientemente importadas

What version of the product are you using? On what operating system?
v13.01

Original issue reported on code.google.com by s...@saulogil.com.ar on 9 Sep 2013 at 1:02

GoogleCodeExporter commented 9 years ago 
Aclaración en la descripción del ticket: Luego de importar las cuentas, en 
Esquema Contable -> Predeterminados aún se ven las cuentas que vienen con la 
distribución de Libertya, no las recientemente importadas.

Esto pasa por que el proceso de actualización de cuentas por default, espera 
tener ciertos flags seteados en i_elementvalue, los cuales no son seteados nunca

Adjunto un parche simple para resolver este problema. 

Un tema a mejorar, luego de aplicar el parche, se informan 32 errores luego de 
la importación, lo cual parece una falsa alarma. Pareciera que al siguiente 
fragmento:

                            if( C_ValidCombination_ID != newC_ValidCombination_ID ) {
                                sql = new StringBuffer( "UPDATE " ).append( TableName ).append( " SET " ).append( ColumnName ).append( "=" ).append( newC_ValidCombination_ID ).append( " WHERE C_AcctSchema_ID=" ).append( C_AcctSchema_ID );

                                int no = DB.executeUpdate( sql.toString(), get_TrxName());

                                log.fine( "ImportAccount.updateDefaultAccount - #" + no + " - " + TableName + "." + ColumnName + " - " + C_ElementValue_ID + " -- " + C_ValidCombination_ID + " -> " + newC_ValidCombination_ID );

                                if( no == 1 ) {
                                    retValue = UPDATE_YES;
                                }
                            }

se le puede agregar

                            }else{
                                retValue = UPDATE_SAME;
                            }

aún no tuve tiempo de ver esto en detalle.

Original comment by s...@saulogil.com.ar on 9 Sep 2013 at 1:11

Attachments:

GoogleCodeExporter commented 9 years ago 
Issue aceptado, pendiente de resolución

Original comment by federico...@disytel.net on 19 Feb 2014 at 3:25