Unverified Commit 173b326a authored by Marius van den Beek's avatar Marius van den Beek Committed by GitHub
Browse files

Merge pull request #17901 from mvdbeek/fix_workflow_saving

[23.2] Fix saving workflows with freehand_comments only
parents a7398dfb b825f798
Loading
Loading
Loading
Loading
+9 −8
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@
workflow steps.
"""
import math
from itertools import chain

from galaxy.util.topsort import (
    CycleError,
@@ -57,6 +58,7 @@ def order_workflow_steps(steps, comments):
                else:
                    sortable_comments.append(comment)

            if sortable_comments:
                # consider comments to find normalization position
                min_left_comments = min(comment.position[0] for comment in sortable_comments)
                min_top_comments = min(comment.position[1] for comment in sortable_comments)
@@ -64,8 +66,7 @@ def order_workflow_steps(steps, comments):
                min_top = min(min_top_comments, min_top)

            # normalize comments by min_left and min_top
            for comment_list in [sortable_comments, freehand_comments]:
                for comment in comment_list:
            for comment in chain(sortable_comments, freehand_comments):
                comment.position = [comment.position[0] - min_left, comment.position[1] - min_top]

            # order by Euclidean distance to the origin