Changing project-organization relationship causes problems

jurgenhaas - October 6, 2009 - 10:41
Project:Storm
Version:7.x-1.x-dev
Component:Storm.module
Category:bug report
Priority:normal
Assigned:Magnity
Status:closed
Description

When I apply for new projects, I initially create a new project in Storm and assign that to a dummy organization. Only if and when I get the order for the project do I create the real organization and reassign that project to this new org, which now is a customer.

Problem is, that all attached notes, etc. still come with the assignment of the correct project and the dummy organization. Which means that those notes are never displayed in Storm anywhere anymore.

This means, when changing the org of a project, all other nodes need to be checked and updated too.

#1

jurgenhaas - October 13, 2009 - 10:53

Is anyone up for this?

#2

Magnity - October 13, 2009 - 13:07

Hi Jurgen,

Personally I've got a lot on at the moment so it would be a case of getting to this (as with all the other issues) when time permits.

If anyone else wants to write a patch that'd be great too. It should be a case of either doing some processing on node_update or firing a hook on node_update for other modules to sort themselves out. I'd prefer the latter though.

#3

homoludens - October 14, 2009 - 12:12
Status:active» needs review

here is the patch for updating organization automatically for tasks, tickets and notes (basically it is the same code). it is not the best solution, but it is a start.

AttachmentSize
stormproject-597050-3_.module.patch 1.53 KB

#4

Magnity - October 15, 2009 - 20:16

Looks like a good start - although i'm a bit confused as to why the $tasks = array(); lines keep popping up.

Obviously we'd need to put something in on update of tasks and tickets too.

#5

Magnity - October 30, 2009 - 22:46
Assigned to:Anonymous» Magnity

Assigning to self - have coded about half of this (live on demo site), and will commit once its all finished.

#6

Magnity - November 8, 2009 - 22:15
Version:6.x-1.x-dev» 7.x-1.x-dev
Status:needs review» patch (to be ported)

Attached patch should fix Drupal 6 branch. Will roll in -dev version from midnight.

Needs porting to Drupal 7.
(Drupal 7 branch is currently unstable but needs to keep up with changes).

AttachmentSize
storm.patch 18.21 KB

#7

Magnity - November 12, 2009 - 11:28
Status:patch (to be ported)» fixed

Committed to D7 (HEAD)

#8

System Message - November 26, 2009 - 11:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.