Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tarea #3337 asignar orden en get line #1572

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

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:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant