[Solved] Possible fixes for this:

[BlackRangerXIII]

This is the map
I need someone to help me test this system made by xYours Trulyx and find any bugs.

Known bugs:
The Line that should be aiming for the crystal is appearing but it's not connecting

 

[KILLCIDE]

Please post the triggers alongside with the map next time!

The issue is in the "Build Start" and "LH Setup" trigger:

Set LH_PointSourceSet = BuildSys_TempLoc Set LH_PointTargetSet = BuildSys_TempLoc2 Trigger - Run LH Build <gen> (ignoring conditions) Custom script: call RemoveLocation(udg_BuildSys_TempLoc) Custom script: call RemoveLocation(udg_BuildSys_TempLoc2)

Set LH_PointSource[LH_MaxIndex] = LH_PointSourceSet Set LH_PointTarget[LH_MaxIndex] = LH_PointTargetSet Trigger - Turn on LH Loop <gen>

He removes the location handle of TempLoc and TempLoc2, thus PointSourceSet, PointTargetSet, PointSource[], & PointTarget[] lose the handle as well. This is why the lightning always appeared to connect in the middle, the X and Y of the locations were either returning as null or 0.

My suggested fix is to do this on index:

Set LH_PointSource[LH_MaxIndex] = (Position of LH_PointSourceSet) Set LH_PointTarget[LH_MaxIndex] = (Position of LH_PointTargetSet)

This should ensure that PointSource[] and PointTarget[] get unique handle ids, so they won't be lost when TempLoc and TempLoc 2 are removed.

Cool system and effects though.

 

[BlackRangerXIII]

Please post the triggers alongside with the map next time!

The issue is in the "Build Start" and "LH Setup" trigger:

Set LH_PointSourceSet = BuildSys_TempLoc Set LH_PointTargetSet = BuildSys_TempLoc2 Trigger - Run LH Build <gen> (ignoring conditions) Custom script: call RemoveLocation(udg_BuildSys_TempLoc) Custom script: call RemoveLocation(udg_BuildSys_TempLoc2)

Set LH_PointSource[LH_MaxIndex] = LH_PointSourceSet Set LH_PointTarget[LH_MaxIndex] = LH_PointTargetSet Trigger - Turn on LH Loop <gen>

He removes the location handle of TempLoc and TempLoc2, thus PointSourceSet, PointTargetSet, PointSource[], & PointTarget[] lose the handle as well. This is why the lightning always appeared to connect in the middle, the X and Y of the locations were either returning as null or 0.

My suggested fix is to do this on index:

Set LH_PointSource[LH_MaxIndex] = (Position of LH_PointSourceSet) Set LH_PointTarget[LH_MaxIndex] = (Position of LH_PointTargetSet)

This should ensure that PointSource[] and PointTarget[] get unique handle ids, so they won't be lost when TempLoc and TempLoc 2 are removed.

Cool system and effects though.

where exactly is that which I should fix

 

[KILLCIDE]

It's in the "LH Setup" trigger.

 

[BlackRangerXIII]

I have a problem
I can't set it to be the position of unit trigger as the LH_PointSourceSet and LH_PointTargetSet are both points not units

 

[KILLCIDE]

You're supposed to be changing PointSource[] and PointTarget[], not PointSourceSet and PointTargetSet. Regardless, that shouldn't be an issue because "Position of Unit" is a point, not a unit.

 

[BlackRangerXIII]

You're supposed to be changing PointSource[] and PointTarget[], not PointSourceSet and PointTargetSet. Regardless, that shouldn't be an issue because "Position of Unit" is a point, not a unit.

Those two are points not units

 

[KILLCIDE]

I know they are two points. I don't understand the issue you have...? Im telling you to cache (Position of Unit), which is a point, into PointSource[] and PointTarget[], which are also points.

EDIT: Baha my mistake. I see what you mean, this is what I meant:

Set LH_PointSource[LH_MaxIndex] = (Position of "UNIT THAT ISSUED BUILD ORDER") Set LH_PointTarget[LH_MaxIndex] = (Position of "STRUCTURE THAT IS BEING BUILT")

I'm not by the computer, so I can't remember the variable names. Something like that though.

 

[BlackRangerXIII]

Explain this please

 

[KILLCIDE]

I don't know what you want me to explain. I see unit variables.

 

[BlackRangerXIII]

is it better?

 

[KILLCIDE]

Yes that is correct. Refer to this post if you're not sure what unit you should be caching.

 

[BlackRangerXIII]

Thanks