Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-4897

Setting the reservation of persons on a routing task isn't reflected in the overview of routing tasks

    Details

    • Sprint:
      Bug Crush Event - 21/2/2015, Community Day 1 - 2015

      Description

      When having set the reservation of persons on a new routing task and the task is saved, the number of persons reserved isn't reflected in the overview of routing tasks.

      1. OFBIZ-4897-14.12.patch
        7 kB
        Gil Portenseigne
      2. OFBIZ-4897.patch
        7 kB
        Gil Portenseigne

        Activity

        Hide
        gil portenseigne Gil Portenseigne added a comment -

        In this patch i add a control on reservPersons and add it to the entry map of updateWorkEffort services.

        Being zealous, i'm adding specific labels, existing one where not good for the field they were checking.

        Should have I given separates patches ?

        Show
        gil portenseigne Gil Portenseigne added a comment - In this patch i add a control on reservPersons and add it to the entry map of updateWorkEffort services. Being zealous, i'm adding specific labels, existing one where not good for the field they were checking. Should have I given separates patches ?
        Hide
        gil portenseigne Gil Portenseigne added a comment -

        same for 14.12 branch

        Show
        gil portenseigne Gil Portenseigne added a comment - same for 14.12 branch
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Gil, Pierre,

        Your patch is in
        trunk r1668314
        R14.12 r1668315
        R13.07 r1668317
        R12.04 r1668316

        Gil you did not need to provide a patch for R14.12. Most of the time "svn merge" is able to automatically do the work (here down to R12.01) and when it fails it's easy to postpone and clear conflicts by hand

        As for the labels, we consider them harmless and allow them to be backported (when well done). So I did.

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Gil, Pierre, Your patch is in trunk r1668314 R14.12 r1668315 R13.07 r1668317 R12.04 r1668316 Gil you did not need to provide a patch for R14.12. Most of the time "svn merge" is able to automatically do the work (here down to R12.01) and when it fails it's easy to postpone and clear conflicts by hand As for the labels, we consider them harmless and allow them to be backported (when well done). So I did.

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            pfm.smits Pierre Smits
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile