-
Notifications
You must be signed in to change notification settings - Fork 221
Conversation
Added three quests, intro to teamwork, intermediate teamwork, and advanced teamwork.
fixed lua style and cleaned up code
changed charVar to localVar
changed getCharVar to getLocalVar
fixed party member requirements
fixed party member requirements
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to have you here with us at Project Topaz, scorchedxi!
I'm just requesting some small changes~!
--player:startEvent(129) -- starts the ready check for all three quests | ||
--player:startEvent(130) -- post third quest dialog | ||
--player:startEvent(131) -- Same job | ||
--player:startEvent(132) -- You don't have the requirements to start the second quest | ||
--player:startEvent(133) -- Same race |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If these events are all implemented into the script now, we no longer need the comments describing them as separate lines~! 😉
(But do keep the comments describing them when you start them later in the file 😄 )
end | ||
end | ||
|
||
function onEventUpdate(player, csid, option) | ||
-- csid 129 happens for both quests | ||
if csid == 129 then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there an outgoing event option that the player selects to confirm that they're ready for the check?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Believe it or not, both options are 0. The 'automated message' packet is what differentiates the response.
|
||
-- check race for second | ||
if (player:getRace() == tpz.race.HUME_M or player:getRace() == tpz.race.HUME_F) and (member:getRace() == tpz.race.HUME_M or member:getRace() == tpz.race.HUME_F) then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could save a few CPU cycles (and line space) by assigning locals of the player's race (right after the party size requirement succeeds) and the member's race (inside the for loop), and then check those cached values in the if instead of calling member:getRace()
each time:
local mRace = member:getRace()
if (pRace == tpz.race.HUME_M or pRace == tpz.race.HUME_F) and (mRace == tpz.race.HUME_M or mRace == tpz.race.HUME_F) then
Fixed spacing and comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, scorchedxi! Happy to have you here with us at Project Topaz!
Added three quests, intro to teamwork, intermediate teamwork, and advanced teamwork.
I affirm: