Re: [Planner Dev] Fix for Altering resource assignments does not trigger save reminder (bugzilla 138368)



On tor, 2004-04-15 at 05:36 +0100, Lincoln Phipps wrote:
> http://bugzilla.gnome.org/show_bug.cgi?id=138368
> 
> See the patch - it has the cludge to get over fact that object->priv->project
> doesn't exist for assignments so calling mrp_object_changed with an assignment
> as your object won't work. Only notice this for setting units.
> 
> FIXME: Should assignments->priv->project actually be expected to
> exist. Is there more to do in that we should also have an
> assignment->priv->project or am I misreading this bit of code in
> mrp_object_changed()

Yeah, I think we should make assignments have a project pointer. It
should be pretty straight-forward, we set the pointer when an assignment
is inserted in the project. This might interfere a bit with the undo
code though.

The fix is a good workaround for now, thanks for digging into it :)
/Richard

-- 
Imendio HB, http://www.imendio.com/




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]