[Health-dev] [bug #46631] Assignation in create prescription moves

[Cédric Krier]

URL:
  <http://savannah.gnu.org/bugs/?46631>

                 Summary: Assignation in create prescription moves
                 Project: GNU Health
            Submitted by: cedk
            Submitted on: Tue 08 Dec 2015 08:56:18 AM CET
                Category: None
                Severity: 2 - Minor
              Item Group: None
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
                 Release: None
                  Module: health_stock

    _______________________________________________________

Details:

In the wizard, after creating the moves, it call Move.assign and then
Move.do.
Why not call only Move.do, I don't see any point to call Move.assign.

Also there could be a small optimisation by calling Move.create only once for
all lines.

If you agree, I can submit a patch.




    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?46631>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/