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

Jcn 449 skip modified data #25

Merged
merged 3 commits into from
Sep 21, 2023
Merged

Jcn 449 skip modified data #25

merged 3 commits into from
Sep 21, 2023

Conversation

mdanelutti
Copy link
Contributor

Este es el primer ajuste que se requiere para poder omitir los datos automáticos al usar update, en este paquete afecta tanto a dateModified como a userModified se agrego una flag nueva para tener control sobre esto, y va de la mano con el package MongoDB, dado que sin importar lo que pase en model, este también suma dateModified.

El package ya paso por pruebas y fueron correctas comento el escenario que se uso para probar:
En OMS se hizo una lambda que actualiza dos datos puntuales a varios pedidos al mismo tiempo por medio de un filtro, para este ejemplo pongamos tres pedidos.

  • Sin la flag ósea funcionamiento normal de siempre podemos darle al update 100 veces que siempre va actualizar los tres pedidos, esto pasa porque en todos los casos lo único que se termina actualizando son los dos campos automáticos nombrados anteriormente.
  • Con flag y pedidos previamente actualizados, para este caso el resultado es siempre que la cantidad de documentos afectados es 0, con lo cual pasa a ser correcto y a que no se actualiza nada de forma automática
  • Con flag y pedidos sin el dato nuevo, para este caso lo que se hizo fue a dos de los tres pedidos borrarles el dato que suma el update y con esto conseguir otro escenario, acá el resultado fue de dos documentos afectados.

Con lo cual el funcionamiento fue el esperado ya que para el que no use la flag sigue todo igual.

@jormaechea jormaechea merged commit a1eb7a3 into master Sep 21, 2023
4 checks passed
@jormaechea jormaechea deleted the JCN-449-skip-modified-data branch September 21, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants