Great Army 1.4

Moderator · Oct 19, 2012

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

05:14, 20th Oct 2012
Magtheridon96: Currently, this spell is not MUI and contains some leaks such as:

((Position of Target) offset by 200.00 towards 0.00 degrees)

and

(Position of Target)

(In the loop trigger)
(These aren't the only leaks though)

Here is a list of tutorials that may help:

edit
Changes aren't made completely.
Something like ((Position of Target) offset by 200.00 towards 0.00 degrees) creates two locations, meaning you need to store the position of the target in one variable, and then have another point set to (TargetPosition offset by 200.00 towards 0.00 degrees) so that you can destroy both of them.

Since it seems that you're having problems with making the correct changes, I'd totally suggest creating a thread here in order to get more people to help you make the spell MUI.

defskull · Oct 19, 2012

Have you tried using this spell for 2 units to cast it at the same time ?
Meaning that you have 2 units that has this spell, each of them cast at different target, but at the same time (with a delay of 1 second is also accepted).

You should try it.

fareezizfar · Oct 19, 2012

defskull said:
Have you tried using this spell for 2 units to cast it at the same time ?
Meaning that you have 2 units that has this spell, each of them cast at different target, but at the same time (with a delay of 1 second is also accepted).

You should try it.

i'll try make the spell

Adiktuz · Oct 19, 2012

lacks configuration triggers and isn't MUI [see defskull's post]...

doomhammer99 · Oct 19, 2012

Not MUI

Else - Actions
- Set Target = (Random unit from (Units within 250.00 of (Position of Target)))
- Unit - Create 1 Rifleman for (Owner of Caster) at ((Position of Target) offset by 200.00 towards 0.00 degrees) facing Default building facing degrees
- Set Point = (Position of (Last created unit))
- Set Spawn = (Last created unit)
- Custom script: call RemoveLocation(udg_Point)

Remove this.
it creates a dummy on the position of the dead bodies.
:thumbs_down:

fareezizfar · Oct 19, 2012

doomhammer99 said:
Not MUI

Else - Actions

Set Target = (Random unit from (Units within 250.00 of (Position of Target)))

Unit - Create 1 Rifleman for (Owner of Caster) at ((Position of Target) offset by 200.00 towards 0.00 degrees) facing Default building facing degrees

Set Point = (Position of (Last created unit))

Set Spawn = (Last created unit)

Custom script: call RemoveLocation(udg_Point)

Remove this.
it creates a dummy on the position of the dead bodies.

Changed to

Else - Actions
- If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  - If - Conditions
    - (Target is Dead) Equal to True
  - Then - Actions
    - Unit Group - Pick every unit (Units within 250.00 of (Position of Target) matching (((Picked unit) is alive) Equal to True)) and do (Actions)
      - Loop - Actions
        
        Set Target = (Picked unit)
        
        Unit - Create 1 Rifleman for (Owner of Caster) at ((Position of Target) offset by 200.00 towards 0.00 degrees) facing Default building facing degrees
        
        Set Point = (Position of (Last created unit))
        
        Set Spawn = (Last created unit)
        
        Custom script: call RemoveLocation(udg_Point)

doomhammer99 · Oct 19, 2012

OH, C'mon. Make it MUI

fareezizfar · Oct 19, 2012

doomhammer99 said:
OH, C'mon. Make it MUI

you're an expert right,, feel free to edit it bro, i don't know what is GUI, MUI, JASS etc... i'm newbies

Almia · Oct 19, 2012

Lol,doomhammer is also a beginner

doomhammer99 · Oct 19, 2012

Radamatus is right.
I'm a beginner too LOL..
If I didn't stop making spells.
I'm an Expert right now

fareezizfar · Oct 19, 2012

Ramadantus: thanks for you help
doomhammer99: i thought that you're expert cuz u know what is MUI

defskull · Oct 19, 2012

Knowing something without knowing how to do it, does not makes you an expert.
Also, "expert" is abstract, you can't really "measure" it.

Also, to post resource (Spells / Systems), it should be MUI at all times.
If your spell is not MUI, you should not post it, because it's against the rules.
Well, rules are not meant to be broken, please follow it.

My Advice: Learn how to make MUI triggers first before posting another spell. If you have learned how to make spells MUI, edit this spell first.

mckill2009 · Oct 19, 2012

Hmm, let me see:
Preview screen - 10/10
Screenshot - 10/10
Description - 5/10 lacks
Readability - 10/10
Usefulness - 5/10
Triggering - -50/10 coz of non-MUI and leaks

Score: -10, pls read tutorials and spell rules...voting for rejection

Almia · Oct 20, 2012

Woah!mckill no no,just help him.
Still the spell is not MUI.
not rejection,but needs fix

fareezizfar · Oct 20, 2012

Updated!!! :goblin_yeah:

mckill2009 · Oct 20, 2012

no matter how you update, as long as this is not MUI, it will be rejected, see the moderator & post #9's recommendation on MUI tutorials...

doomhammer99 · Oct 20, 2012

LOL..don't put my name. I'm still fixing your spell..
I'll send it to you later..

Magtheridon96 · Oct 20, 2012

Why are you constantly resetting the status to Pending if 0 changes are being made?

defskull · Oct 20, 2012

He wants to escape the "Needs fix" ?

Magtheridon96 · Oct 20, 2012

1 more attempt at updating without making any changes just to escape a "Needs fix", and I'll make that "Needs fix" a "Rejected".

fareezizfar · Oct 21, 2012

nowhere in my mind to escape from need fix.. i only going to change the description..

BUP · Nov 8, 2012

I'm a Newbie tooo....
My Join Date is later than yours
But my world editor experience is six months now.
So far, I made one MUI spell

Great Army 1.4

Great Army by ~.:KySkE:.~ (Map)

Moderator

Moderator

Similar threads