Skip to content
Snippets Groups Projects
Commit 6435d0a5 authored by Zoey76's avatar Zoey76
Browse files

Datapack code refactoring:

	* Removed "Loaded quest ...", with 600 quest this will be more likely spam.
	* Changed getQuestItemsCount(itemId) == 0 and getQuestItemsCount(itemId) < 1 for !st.hasQuestItems(itemId).
	* Changed getQuestItemsCount(itemId) > 0 and getQuestItemsCount(itemId) >= 1 for st.hasQuestItems(itemId).
	* Changed st.getState() != State.COMPLETED and st.getState() == State.COMPLETED for !qs.isCompleted() and qs.isCompleted() respectively.
	* Added some JavaDocs to event mods and minor code improvement.
	* Changed switch statements with one case for if statement.
	* Changed takeItems(itemId, st.getQuestItemsCount(itemId)) for takeItems(itemId, -1) where it was possible.
	* Changed for(int npcId : NPCIDS) addKillId(npcId); for addKillId(NPCIDS); can be done in more places.
	* Unhardcoded class Id in Q401_PathToWarrior.
parent 114aeccb
No related branches found
No related tags found
No related merge requests found
Showing
with 55 additions and 54 deletions
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment