- Few optimization/fix done for Stolen Dignity (386)

(require core)

Reported by: hellEVIL-13

Reported by: u3games

Thanks to @Sacrifice78

-- Fixed License in file (L2J Server by L2J DataPack)
-- Removed QuestState import, not's needed
-- Added a minimal description of script
-- Fixed spawns of NPC to retail values. Added missing spawn in Heine.
Thanks to @maneco2 for coordinates
-- Added comments where appropriate
-- Removed ANTHARAS array and his "for each" loop. Thanks to @maneco2
for his comment at respect.
-- Moved SPAM array
-- Edited onFirstTalk() method
-- Removed check condition in onFirstTalk() method

Reported By: @Sdw, @maneco2
Patch By: @Sacrifice
Tested By: @Sacrifice

This commit implements the Nevits Herald npc. It gives the Nevits
Hourglass buff to players.

Patch By: @Sacrifice
Tested By: @Sacrifice

This commit fixes collision height and radius for the Kadomas and Flame
Box npcs.

Reported By: @u3games
Patch By: @u3games
Reviewed By: @Sacrafice, @LasTravel
Tested By: @u3games, @Sacrifice

This commit fixes a bug where you can banish castle foreigners during
territory war.

Reported By: @l2mmo3d
Patch By: @l2mmo3d

Using super to get the this reference as a reference of the type of the
super class is not valid. According to java documentation super can be
used to access members and methods of it's super class. Using it to get
a reference to the current class is not documented. In the past ECJ
allowed this expression and hapilly compiled. But tests one year ago has
shown that the JDK compiler in tools.jar was reporting it as an error.
Now as reported @Sacrifice, the last update of Eclipse also complains
about this, so this expression is now removed and the code a little bit
cleaned using a lambda instead of inner class.

Reported By: @Sacrifice
Patch By: @Sacrifice, @HorridoJoho

Patched by: @UnAfraid

Patched by: @valdaron

Fix #316

Reported by: @Affinity90

Fixed minor typos.

Static comparator to prevent initialization on each call.
List is created after single-target checks are performed.
Using LinkedList instead ArrayList due resize overhead.
Adding target only if it's not null.
Sorting the list of creatures after it's complete, not after adding the
target (that wouldn't have an effect on the list since it has a single
element).
Performing maximum target check first.
Skipping hidden creatures.
Skipping, when in siege, creatures that are not in the same siege side.

targeted

Fix #293

Minor typo in quest Trial of Duty (212)

Reported by: @ghost

Update Q00227_TestOfTheReformer.java

Missing HTML page: data/scripts/quests/Q00212_TrialOfDuty/30109-04.html