Tarea #3337 asignar orden en get line #1572
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Descripción
Las líneas de documentos de compra y venta tienen un campo orden, que usamos para ordenar las líneas. Este campo lo inicializamos a cero y luego en javascript lo rellenamos. Pero si en la función getNewLine() implementamos un contador de líneas y rellenamos el campo orden, nos ahorramos tener que usar el javascript.
Eliminamos el javascript que asigna el orden al guardar. Se mantienen las llamadas a la funcion ordenar en javascript cuando se ordena con el jquery-ui sortable.
Se crea un método que devuelve el ultimo orden para una nueva línea. En este caso se obtiene el valor mínimo y se resta 10 para mantener la retrocompatibilidad, ya que desde siempre se a asignado el orden de las líneas decrecientemente.
¿Cómo has probado los cambios?
Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado: