{"action":"created","issue":{"url":"https://api.github.com/repos/u3games/L2J_EventEngine/issues/104","labels_url":"https://api.github.com/repos/u3games/L2J_EventEngine/issues/104/labels{/name}","comments_url":"https://api.github.com/repos/u3games/L2J_EventEngine/issues/104/comments","events_url":"https://api.github.com/repos/u3games/L2J_EventEngine/issues/104/events","html_url":"https://github.com/u3games/L2J_EventEngine/pull/104","id":116613660,"number":104,"title":"Avoid a NPE when you interact with npc that doesn't have NpcHolder","user":{"login":"luksdlt92","id":4108820,"avatar_url":"https://avatars.githubusercontent.com/u/4108820?v=3","gravatar_id":"","url":"https://api.github.com/users/luksdlt92","html_url":"https://github.com/luksdlt92","followers_url":"https://api.github.com/users/luksdlt92/followers","following_url":"https://api.github.com/users/luksdlt92/following{/other_user}","gists_url":"https://api.github.com/users/luksdlt92/gists{/gist_id}","starred_url":"https://api.github.com/users/luksdlt92/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/luksdlt92/subscriptions","organizations_url":"https://api.github.com/users/luksdlt92/orgs","repos_url":"https://api.github.com/users/luksdlt92/repos","events_url":"https://api.github.com/users/luksdlt92/events{/privacy}","received_events_url":"https://api.github.com/users/luksdlt92/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"luksdlt92","id":4108820,"avatar_url":"https://avatars.githubusercontent.com/u/4108820?v=3","gravatar_id":"","url":"https://api.github.com/users/luksdlt92","html_url":"https://github.com/luksdlt92","followers_url":"https://api.github.com/users/luksdlt92/followers","following_url":"https://api.github.com/users/luksdlt92/following{/other_user}","gists_url":"https://api.github.com/users/luksdlt92/gists{/gist_id}","starred_url":"https://api.github.com/users/luksdlt92/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/luksdlt92/subscriptions","organizations_url":"https://api.github.com/users/luksdlt92/orgs","repos_url":"https://api.github.com/users/luksdlt92/repos","events_url":"https://api.github.com/users/luksdlt92/events{/privacy}","received_events_url":"https://api.github.com/users/luksdlt92/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2015-11-12T18:42:21Z","updated_at":"2015-11-18T14:20:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/u3games/L2J_EventEngine/pulls/104","html_url":"https://github.com/u3games/L2J_EventEngine/pull/104","diff_url":"https://github.com/u3games/L2J_EventEngine/pull/104.diff","patch_url":"https://github.com/u3games/L2J_EventEngine/pull/104.patch"},"body":"## Summary\r\n\r\n[ENG] Avoid a NPE when you interact with npc that doesn't have NpcHolder. For example, you could talk with Event Engine Manager when you are waiting to teleport, then the NPE appears.\r\n\r\n[ESP] Evita un NPE que sucede cuando interactuas con un NPC que no tiene NpcHolder. Por ejemplo, podrías hablar con el Event Engine Manager cuando estás esperando para ser teletransportado y el NPE aparecería."},"comment":{"url":"https://api.github.com/repos/u3games/L2J_EventEngine/issues/comments/157726498","html_url":"https://github.com/u3games/L2J_EventEngine/pull/104#issuecomment-157726498","issue_url":"https://api.github.com/repos/u3games/L2J_EventEngine/issues/104","id":157726498,"user":{"login":"luksdlt92","id":4108820,"avatar_url":"https://avatars.githubusercontent.com/u/4108820?v=3","gravatar_id":"","url":"https://api.github.com/users/luksdlt92","html_url":"https://github.com/luksdlt92","followers_url":"https://api.github.com/users/luksdlt92/followers","following_url":"https://api.github.com/users/luksdlt92/following{/other_user}","gists_url":"https://api.github.com/users/luksdlt92/gists{/gist_id}","starred_url":"https://api.github.com/users/luksdlt92/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/luksdlt92/subscriptions","organizations_url":"https://api.github.com/users/luksdlt92/orgs","repos_url":"https://api.github.com/users/luksdlt92/repos","events_url":"https://api.github.com/users/luksdlt92/events{/privacy}","received_events_url":"https://api.github.com/users/luksdlt92/received_events","type":"User","site_admin":false},"created_at":"2015-11-18T14:20:17Z","updated_at":"2015-11-18T14:20:17Z","body":"@fissban Ahí quedó mucho mejor. Básicamente el problema era que sólo retornaba true el onInteract cuando el jugador no estaba en el evento y el npc tampoco. Pero en el caso que el jugador esté esperando para ser teletransportado (por lo tanto, está en el evento) y el NPC no (como es el caso del Manager), seguía de largo y generaba el NPE.\r\n\r\nCon cambiar un && por || ya está.\r\n\r\nRespecto a ver cómo está programado el NPC, creo que se puede ver en otro PR.\r\n\r\n¿Qué pensás?"}}