Moderator
M
Moderator
11:57, 22nd Jun 2009
Coding: 2.5/5
The coding was alright (although quite simple), but you should really remove the 'Do Nothing' action. Global variables don't need to be nulled at the end of the trigger. On that note, you don't even need your SparkyAuraX variables. You also might want to prevent it from damaging allies (while using a spell such as holy light, heal, etc...). Support for multiple levels could have also improved the spell quite a bit.
Originality: 4/5
It's pretty original; being able to amplify the damage of any other spell cast (well, sorta).
Documentation: 4/5
An acceptable amount of documentation was used, although the grammar was off at times.
Overall = (Coding * 0.50) + (Originality * 0.30) + (Documentation * 0.20)
Overall: 3.25
Approved with a 3/5 (Useful) rating.
Coding: 2.5/5
The coding was alright (although quite simple), but you should really remove the 'Do Nothing' action. Global variables don't need to be nulled at the end of the trigger. On that note, you don't even need your SparkyAuraX variables. You also might want to prevent it from damaging allies (while using a spell such as holy light, heal, etc...). Support for multiple levels could have also improved the spell quite a bit.
Originality: 4/5
It's pretty original; being able to amplify the damage of any other spell cast (well, sorta).
Documentation: 4/5
An acceptable amount of documentation was used, although the grammar was off at times.
Overall = (Coding * 0.50) + (Originality * 0.30) + (Documentation * 0.20)
Overall: 3.25
Approved with a 3/5 (Useful) rating.