Code review comment for lp:~otherway/sepa-tools/sepa-tools

Revision history for this message
Pedro Rodríguez Gil (Otherway) (pedro-q) wrote :

I didn't removed the mandate sequence, just replaced the default value for mandate_unique field with a slash and added the sequence on create because if set on default value it is getting a new sequence each time you enter the creation form and if discard the sequence is lost, moreover, if you save it is getting a new sequence again, i.e., if the next sequence is RUM00001 when you enter the creation it is got for default value but on save it takes a new sequence again and the mandate is saved with sequence RUM00002. Next one will be saved with RUM00004 and so on.

Hope this explains your answer.

Greetings.

Atentamente,
Pedro Rodríguez Gil

Otherway Creatives, S.L.
<email address hidden>
(+34) 957 000 433
(+34)682 622 967

-----------------------------------------------------------------
Este correo y sus archivos asociados son privados y confidenciales, y va dirigido exclusivamente a su destinatario. Si recibe este correo sin ser el destinatario del mismo, le rogamos proceda a su eliminación y lo ponga en conocimiento del emisor. La difusión por cualquier medio del contenido de este correo podría ser sancionada conforme lo previsto a las leyes españolas. No se autoriza la utilización con fines comerciales o para su incorporación a ficheros automatizados de las direcciones del emisor o del destinatario.

El 28/02/2014, a las 10:54, "Ignacio Ibeas \(www.acysos.com\)" <email address hidden> escribió:

Review: Needs Information

Hello,

What is the reason to remove the mandate sequence?

Greetings
--
https://code.launchpad.net/~pedro-q/sepa-tools/sepa-tools/+merge/208136
You are the owner of lp:~pedro-q/sepa-tools/sepa-tools.

« Back to merge proposal