Need help with a quest!

[Weirdoze]

whats wrong with this?

deliver magic shard
Events Unit - A unit comes within 400.00 of (Get last order target unit of Alchemist 0039 <gen>)
Conditions
(Item carried by (Triggering unit) of type |c0020c000Magic shard|r) Equal to (Item carried by (Triggering unit) of type |c0020c000Magic shard|r)
Actions
Item - Remove (Item carried by (Triggering unit) of type |c0020c000Magic shard|r)
Game - Display to (Triggering player), at offset (0.00, 0.00) for 30.00 seconds the text: |c00cccc00Alchemist...
Hero - Add 75 experience to (Triggering unit), Hide level-up graphics
Player - Add 25 to (Owner of (Triggering unit)) Current gold
Quest - Mark (Last created quest) as Completed
Quest - Display to (All players) the Quest Completed message: |c0020c000Quest com...

deliver magic shard Complete
Events
Unit - A unit comes within 400.00 of (Get last order target unit of Mage 0015 <gen>)
Conditions
Actions
Game - Display to (Triggering player), at offset (0.00, 0.00) for 30.00 seconds the text: |c00cccc00Matt|r: H...
Hero - Give (Item carried by Mage 0015 <gen> of type |c0020c000Magic shard|r) to (Triggering unit)
Quest - Create a Optional quest titled Deliver magic shard with the description Matt wants you to d..., using icon path ReplaceableTextures\CommandButtons\BTNGem.blp
Quest - Display to (All players) the Quest Update message: |c00fffc01Quest dis...
Set Delivermagicshard = (Last created quest)

plz tell whats wrong..

 

[Weirdoze]

Neither of them works in editor "ERROR"

 

[Kazuga]

I have written some comments inside of it..

deliver magic shard Complete
Events
Unit - A unit comes within 400.00 of (Get last order target unit of Mage 0015   <gen>)  (Eeh.. do you actually know what you have done here? You have done the event a unit comes within 400 of your targeted unit" not a specific unit. This means if you target yourself this trigger will go off.... Instead I suggest you pick the questgiver or whatever the unit is that is going to trigger the trigger...)
Conditions  (Here I suggest you take a condition of a player or hero or whatever, otherwise a normal creep could trigger it, or any sort of unit)
Actions
Game - Display to (Triggering player), at offset (0.00, 0.00) for 30.00 seconds the text: |c00cccc00Matt|r: H...  (Display text at offset? Wtf?? Instead take "Display to owner of unit (Triggering unit) the text :------)
Hero - Give (Item carried by Mage 0015 <gen> of type |c0020c000Magic shard|r) to (Triggering unit)
Quest - Create a Optional quest titled Deliver magic shard with the description Matt wants you to d..., using icon path ReplaceableTextures\CommandButtons\BTNGem.blp
Quest - Display to (All players) the Quest Update message: |c00fffc01Quest dis...
Set Delivermagicshard = (Last created quest)

 

[Nina]

deliver magic shard
Events Unit - A unit comes within 400.00 of (Get last order target unit of Alchemist 0039 <gen>)
Conditions
[B](Item carried by (Triggering unit) of type |c0020c000Magic shard|r) Equal to (Item carried by (Triggering unit) of type |c0020c000Magic shard|r)[/B]Actions
Item - Remove (Item carried by (Triggering unit) of type |c0020c000Magic shard|r)
Game - Display to (Triggering player), at offset (0.00, 0.00) for 30.00 seconds the text: |c00cccc00Alchemist...
Hero - Add 75 experience to (Triggering unit), Hide level-up graphics
Player - Add 25 to (Owner of (Triggering unit)) Current gold
Quest - Mark (Last created quest) as Completed
Quest - Display to (All players) the Quest Completed message: |c0020c000Quest com...

The black part condition its really weird make it maybe a boolean like (Item carried by (Triggering unit) of type |c0020c000Magic shard|r) Equal to true, coz you are using (Item carried by (Triggering unit) of type |c0020c000Magic shard|r) Equal to (Item carried by (Triggering unit) of type |c0020c000Magic shard|r) which both values are the same in EVERY CASE so it would never work or always work, if not use a boolean make it (Item carried by (Triggering unit) equal to |c0020c000Magic shard|r

 

[Weirdoze]

ill try those Thank you