As per title.

Comments

esoteric1’s picture

Assigned: Unassigned » esoteric1
esoteric1’s picture

Assigned: esoteric1 » Unassigned
Status: Active » Needs work

After having looked at this, I think it would be really good if someone who has worked on this or knows how this works makes a few comments before I start ripping out the old d6 stuff.

Specifically, javascript-related bits like this:

function _stormtask_project_assignments_js($organization_nid, $project_nid) {
  $assignments = storm_get_assignment_options($organization_nid, $project_nid);
  print drupal_to_js($assignments);
  exit();
}

If someone can give some insights and guidelines on how it was in d6 and what we want in d7, I can then fix it up according to what we want it to do in d7.

juliangb’s picture

The best way in D7 seems to be using the #ajax method within Form API.

You can see examples of this in the task node form, and note node form (when I next upload a patch).

juliangb’s picture

Project: Storm » Drupal PM (Project Management)
Component: Storm Task » Code

Moving to the new Project Management module.

juliangb’s picture

Status: Needs work » Needs review
StatusFileSize
new19.4 KB

Attaching a patch to test.

Haven't included the category and status fields with this iteration due to the problems with the views handlers at the moment - let's test this and then add those in later (just editing an existing view should be easier for anyone to do)

juliangb’s picture

StatusFileSize
new26.8 KB

This should hopefully include the new views default file.

Status: Needs review » Needs work

The last submitted patch, pm-tasks-views-2.patch, failed testing.

juliangb’s picture

Status: Needs work » Needs review
StatusFileSize
new26.88 KB

Amended patch.

juliangb’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit a8f98de on 7.x-1.x, 7.x-2.x by juliangb:
    Issue #1848242 by juliangb: Change storm/tasks page to be made by views.