Automatically add Task Items when editing a Task.

[bsaiken]

This Issue will reduce the steps needed to add Task technicians, locations, field parameters (analytes), lab analytes, service providers.
Currently, using the left menu these are automatically added to the Project/PM, but not to the Task. The left menu behaviors will stay the same.

New behaviors (adding each of the above listed items to the Task): Add buttons to each of the listed tabs: /task/technicians --> Add Task Technician /task/locations --> Add Task Location /task/fieldAnalytes --> Add Field Parameter /task/labAnalytes --> Add Lab Parameter /task/services --> Add Service Provider

When clicking the Add Task Technician button, the technician will be added to the database, in the same manner as using the left menu, but the technician will also be added to the task_technician table, as is done when clicking the button value="Add technician(s)".

I think the easiest way to do this is have the "Add Task Technician" button direct the user to the page technicain/showForm, except that the url will be /technician/showForm?task_id=#. Then, at line 164 of technician_manager.js add code some thing like this: if (isset(task_id)) {call the function updateTechnicians with the new technician_id}

Then redirect to /task/technicians. The newly added Technician should appear in the left hand column. If the task_id is not set, the function will continue as normal (redirecting to technician/listAll).

This method (or similar) will be used for all identified instances. Whatever method is chosen to accomplish this Issue, do only one instance (technicians) and I can copy the method for all other instances.

[WebDevJL]

@bsaiken

So the main underlying functionality that you want when editing a task and:

• adding a technician, is that we add a row in the tables
  • technician
  • task_technician
• adding a location, is that we add a row in the tables
  • location,
  • project_location
  • task_location
• adding a Field Parameter, is that we add a row in the tables
  • field_analyte
  • task_field_analyte
• adding a Lab Analyte, is that we add a row in the tables
  • lab_analyte
  • task_lab_analyte
• adding a service, is that we add a row in the tables
  • service,
  • project_service
  • task_service

Is that correct?

[bsaiken]

@WebDevJL Yes, I thought this was obvious, so I did not list each one. No harm in clarifying, though.

[WebDevJL]

@bsaiken Done for the technicians. Code committed to feature_PM if you want to test.

[bsaiken]

@WebDevJL Did you mean to assign me to this and change the label to function review?

[WebDevJL]

@bsaiken No. I'm just letting you know that I have done it for the technicians and today for the locations. The rest will come soon.

You can test on feature_PM branch if you need.

[bsaiken]

@WebDevJL I did test on the feature_PM branch and it did not work for technicians.

[WebDevJL]

@bsaiken

Please post a screenshot of chrome dev console with the response after clicking add on the page technician/showForm.

Also do another pull and retest please.

Thank you. On 13 Aug 2015 16:56, "bsaiken" notifications@github.com wrote:

@WebDevJL https://github.com/WebDevJL I did test on the feature_PM branch and it did not work for technicians.

— Reply to this email directly or view it on GitHub https://github.com/FWAJL/FieldWorkManager/issues/1061#issuecomment-130720632 .

[bsaiken]

@WebDevJL I would do that, of course, but there was not response.

[FWAJL]

@bsaiken

It works on that branch. See my screenshots:

If you really can't get it to work, check in the dev tools the tab "Elements" and check if you see a HTML comment that looks like an error logged. It is always at the very top (see rectangle in red below).

[bsaiken]

@FWAJL Just push it to dev