nalc
/
areas
forked from mff/areas
1
0
Fork 1
Code Issues Pull Requests Releases Wiki Activity

2 fixes mineurs #1

Closed
crabman77 wants to merge 0 commits from crabman77/areas:fix_check into master
pull from: crabman77/areas:fix_check
merge into: nalc:master
Conversation 1 Commits - Files Changed -
crabman77 commented 2022-07-09 03:12:17 +02:00
Member
Copy Link

fixe un petit bug introduit quand j'ai crée openfarming, qui ignore le test de faction si une area est openfarming.

Et ne return pas false si ce n'est pas un joueur(ie:mobs) ou si c'est un node inconnu, continue l'itération sur les autres areas.

C'est des bug mineurs, car généralement il n'y a qu'une area donc c'est de toute façon protegé, et pour faction, il faut vraiment tomber sur une area openfarming et 2 joueurs qui pvp dans l'area et fasse partie d'une faction pour que ça echoue.

fixe un petit bug introduit quand j'ai crée openfarming, qui ignore le test de faction si une area est openfarming. Et ne return pas false si ce n'est pas un joueur(ie:mobs) ou si c'est un node inconnu, continue l'itération sur les autres areas. C'est des bug mineurs, car généralement il n'y a qu'une area donc c'est de toute façon protegé, et pour faction, il faut vraiment tomber sur une area openfarming et 2 joueurs qui pvp dans l'area et fasse partie d'une faction pour que ça echoue.
crabman77 added 1 commit 2022-07-09 03:12:17 +02:00
a8cacee8cc Fixed a minor bug introduced when I created openfarming, which skips the faction test if an area is openfarming. 
And not return false if not a player(ie:mobs api) or unknown node, continue iterate on others areas(minor bug, generaly if 1 area it's protected).
sys4 commented 2022-07-09 16:59:35 +02:00
Owner
Copy Link

Fusionné.

Fusionné.
sys4 closed this pull request 2022-07-09 16:59:35 +02:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: nalc/areas#1
No description provided.
Delete Branch "crabman77/areas:fix_check"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?