[calligraplan] [Bug 496133] dependency of task is wrongly calculated when tasks have priority

[miko]

https://bugs.kde.org/show_bug.cgi?id=496133

--- Comment #2 from miko <mickael.serg...@free.fr> ---
Origin of the issue could be in scheduleForward() function. In fact, for
priority it call scheduleFromStartTime which doesn't take into account
predecessors instead of scheduleForward().

We don't find the bug in calculateBackward() which is a mirror function.

Probably a copy/past error from line 1040...1046 ?

Here a possible fix. 

diff --git a/src/libs/kernel/kptproject.cpp b/src/libs/kernel/kptproject.cpp
index 37d612e8..ab58a89a 100644
--- a/src/libs/kernel/kptproject.cpp
+++ b/src/libs/kernel/kptproject.cpp
@@ -1026,7 +1026,7 @@ DateTime Project::scheduleForward(const DateTime
&earliest, int use)
     if (!m_priorityNodes.isEmpty()) {
         for (Node *n : qAsConst(m_priorityNodes)) {
             cs->logDebug(QStringLiteral("Schedule task '%1' by priority:
%2").arg(n->name()).arg(n->priority()));
-            DateTime time = n->scheduleFromStartTime(use); // do not do
predeccessors
+            DateTime time = n->scheduleForward(earliest, use);
             if (time > end) {
                 end = time;
                 cs->setLatestFinish(time);

-- 
You are receiving this mail because:
You are watching all bug changes.