Item System v1.6

[Moderator]

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

14:57, 6th Feb 2010
The_Reborn_Devil:

Sorry, but this is still to simple. Turn it into a system. Currently you have to create a bunch of triggers if you have a lot of heroes on your map. This is both inefficient, space consuming and ugly to look at.


Status: Rejected until updated
Rating: N/A

PM me or another mod once you've updated this to get it reviewed again. Have a nice day!

 

[Newbestyle]

Googlieeyes,

I suppose,

(will be updating it to make sets later)

only means the possibility to customize it more easily.

It is not even clear to me what this system should do...
Either it should stop the heroes from picking up those items or it does not work...

I will take a look at the triggers once again tomorrow. For now I am too tired.

Newbestyle

 

[Googlieeyes]

if a hero who is no assigned to pick it up, then they cannot otherwise get it

 

[China-Man]

Googlieeyes wat map in progress u working on :?

 

[Newbestyle]

Well, at first I have to say sorry for waiting so long with my answer,
I had a lot to do and things to worry about.
Back to topic:

if a hero who is no assigned to pick it up, then they cannot otherwise get it

Okay, now I'm downloading it once again, because the first time it seemed to me NO hero could add NO item of those included in the map.

Maybe I am wrong - so i take a look on it once again.

[Downloading]

[EDIT]
I downloaded it and tried it...
Like the first time NO hero of those can add any of the items.

After trying this I opened it with my World Editor and it said my editor got a lower version than yours.
I dont know wether its responsible for that.

[Updating]

Updated it.
Is working!
Without the update my WC3 can't use that one trigger. ;P
Hm, it's working fine but seems to me as a system which isn't that difficult to make.
4,5 from me.

Sorry, for keep you waiting that long.

 

[cHeRoL]

Good and usefull.

 

[Noobsmoke]

just ... try to do somethink like dual wield , two handed , one hand, off hand, ranged etc ... and weapons category .... mace, sword, dagger ... can u do that ?

 

[Darkzealot_128]

This spell is useful and looks pretty awesome.

4/5

 

[Darkill]

Idiot he is Idiot reair it it drom anything from any hero!!!!!!!!!!!!

 

[Googlieeyes]

Idiot he is Idiot reair it it drom anything from any hero!!!!!!!!!!!!

Excuse me what?

 

[-Kobas-]

To Darkill

Idiot he is Idiot reair it it drom anything from any hero!!!!!!!!!!!!

Please don't insult!

Each hero can pick only items created for his class!
Next time check triggers first before testing to know what this one do!

To Googlieeyes
Make better description pls!
Same for gameplay!
Add some floating text trigger action to show players what happened!

Also Remove Do Nothing Action from here:

• For each (Integer A) from 1 to 6, do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • (Item-type of (Item carried by (Hero manipulating item) in slot (Integer A))) Equal to Searing Blade
            • (Item-type of (Item carried by (Hero manipulating item) in slot (Integer A))) Equal to Ancestral Staff
      • Then - Actions
        • Set AlreadyCarryingEquip = (AlreadyCarryingEquip + 1)
      • Else - Actions
        • Do nothing

There is no need for it!

 

[Googlieeyes]

To Darkill

Please don't insult!

Each hero can pick only items created for his class!
Next time check triggers first before testing to know what this one do!

To Googlieeyes
Make better description pls!
Same for gameplay!
Add some floating text trigger action to show players what happened!

Also Remove Do Nothing Action from here:

• For each (Integer A) from 1 to 6, do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • (Item-type of (Item carried by (Hero manipulating item) in slot (Integer A))) Equal to Searing Blade
            • (Item-type of (Item carried by (Hero manipulating item) in slot (Integer A))) Equal to Ancestral Staff
      • Then - Actions
        • Set AlreadyCarryingEquip = (AlreadyCarryingEquip + 1)
      • Else - Actions
        • Do nothing

There is no need for it!

huh thought i did remove the do nothings. i'll check that again.

Just checked. that was the only one i missed >.< thanks.


Also

Change set 2:
*Fixed a do nothing i missed and removed it.
*Added descriptions to the items to be more understandable about the system.
*Have been playing around with the system if you want me to change it I can make it for helmets, armor, wpn, offhand, misc, and boots/gloves. It would work and limit for just 1 of each. I'm thinking about changing this to be the next update. Let me know if you think this would help.

 

[Prince.Zero]

Item systems are meant to be coded in JASS
This will die in vain

 

[Vlad727]

@ Prince.Zero

Everyone has the right to decide for themself if they wish to use GUI, Jass, or VJass.
Some mapmaker have not yet learned how to use Jass or vJass yet
and therefore decide to use a GUI system.
I think saying that "this system is useless", is utter nonsense.
If you read some of the comments above then you can see that there are people who are indeed happy that this system is available to them.

So in short:
"if you have something against people using GUI, or uploading GUI systems,
then please just leave them be, and return to your jass section of the spells"