@@ -65,22 +65,28 @@ export function NodeViewer({
6565 onUpdateNode ( node . id , updatedNode , "set-bugzilla-number" , true ) ;
6666 } ;
6767
68- const handleAddDependency = ( dependencyId : TechNodeId ) => {
69- // TODO: fix this after inverting the tree
68+ const handleAddBlocks = ( blocksId : TechNodeId ) => {
69+ let blocks = fullTree . nodes . find ( ( x ) => x . id === blocksId ) ;
70+ if ( ! blocks ) {
71+ return ;
72+ }
7073 const updatedNode = {
71- ...node ,
72- dependsOn : [ ...node . dependsOn , dependencyId ] ,
74+ ...blocks ,
75+ dependsOn : [ ...blocks . dependsOn , node . id ] ,
7376 } ;
74- onUpdateNode ( node . id , updatedNode , "add-dependency" , false ) ;
77+ onUpdateNode ( blocksId , updatedNode , "add-dependency" , false ) ;
7578 } ;
7679
77- const handleRemoveDependency = ( dependencyId : TechNodeId ) => {
78- // TODO: fix this after inverting the tree
80+ const handleRemoveBlocks = ( blocksId : TechNodeId ) => {
81+ let blocks = fullTree . nodes . find ( ( x ) => x . id === blocksId ) ;
82+ if ( ! blocks ) {
83+ return ;
84+ }
7985 const updatedNode = {
80- ...node ,
81- dependsOn : node . dependsOn . filter ( ( id ) => id !== dependencyId ) ,
86+ ...blocks ,
87+ dependsOn : blocks . dependsOn . filter ( ( id ) => id !== node . id ) ,
8288 } ;
83- onUpdateNode ( node . id , updatedNode , "remove-dependency" , false ) ;
89+ onUpdateNode ( blocksId , updatedNode , "remove-dependency" , false ) ;
8490 } ;
8591
8692 const handleDelete = ( ) => {
@@ -266,7 +272,7 @@ export function NodeViewer({
266272 </ div >
267273 { editing && (
268274 < button
269- onClick = { ( ) => handleRemoveDependency ( depId ) }
275+ onClick = { ( ) => handleRemoveBlocks ( depId ) }
270276 className = "ml-2 text-red-500 hover:text-red-700"
271277 title = "Remove dependency"
272278 >
@@ -349,7 +355,7 @@ export function NodeViewer({
349355 < NodePickerModal
350356 nodes = { availableNodes }
351357 editing = { editing }
352- onPickNode = { handleAddDependency }
358+ onPickNode = { handleAddBlocks }
353359 onAddNode = { onAddNewDependedOnBy }
354360 onClose = { ( ) => setShowNodePicker ( false ) }
355361 />
0 commit comments