Do you see anything wrong with this quest code?

[reddogut]

sub EVENT_ITEM {
    if(plugin::check_handin(\%itemcount, 13071 => 4)){
    quest::say("This is good work! I know this isn't much, but it is all I can offer you. Thanks for your help!");
    quest::faction(48,10); # +Coalition of Tradefolk Underground
    quest::faction(105,10); # +Freeport Militia
    quest::faction(184,-10); # -Knights of Truth
    quest::faction(258,-10); # -Priests of Marr
    quest::exp(5);
    quest::givecash(1,1,1,0);
 }
}