Quest I made ( needs critiqueing)

[_Kerosh_]

Been looking at your quest and found a few errors.

Remember syntax : ), sub EVENT_SAY has a { correct? Where is the one ending the sub event?

Should be after he last function.

It should be quest::say, not questsay::
if ($text=~/earrings/i){
questsay::("I can craft, you a magical [Hoop of Dark Spirits], All I require is a mithril earing, and a jacnith. I will also require 500pp for my work.");
}
if ($text=~/rings/i){
questsay::("You would like a ring? Well I can craft you a ring of dreadful dreams. For this magical ring I require a dream stone, and a silver emerald ring. I will also require 1000 platinum peices for my work.");
}

Hoop of Dark Spirits, you put brackets around it as if they were supposed to say something, but you stopped there?

sub EVENT_ITEM needs a bracket to close it also
sub EVENT_ITEM
{
}
Also, how I would organize my code is like this:
*Note* There are spaces in there, the forum just deleted them.
sub EVENT_SAY
{
if($text=~/hail/i)
{
quest::say("Hullo.");
}
}
As you can see, it is a lot easier to tell what brackets are matched up with what. Just tab every time you have a new thing, such as

x=0;
sub EVENT_SAY
{
if(text=~/hail/i)
{
if($race == "Dwarf")
{
quest::say("Good dwarf, leave me be.");
}
}
}
Also, don't forget elsif, it is much better for coding:
if()
elsif()
else()

Hope I helped : )