Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FAKE_NPC having a sprite #1916

Closed
3 tasks done
patrickkuhlmann opened this issue Nov 26, 2017 · 5 comments
Closed
3 tasks done

FAKE_NPC having a sprite #1916

patrickkuhlmann opened this issue Nov 26, 2017 · 5 comments
Labels
component:core Affecting the Hercules core (i.e. not the game mechanics directly) severity:3-medium Issues with GM commands, script commands, and other commonly used features

Comments

@patrickkuhlmann
Copy link
Contributor

Issue Prelude

  • I have not modified the source prior to reproducing this issue.
  • I am using the latest version of Hercules.
  • I am aware that this report will be closed or deleted if it becomes obvious that I am stating the false.

Description

The FAKE_NPC is having a sprite with the latest stable version.
This can be seen here

image

image

Operating System

No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 16.04.3 LTS
Release: 16.04
Codename: xenial

@dastgirp dastgirp added component:core Affecting the Hercules core (i.e. not the game mechanics directly) severity:3-medium Issues with GM commands, script commands, and other commonly used features status:confirmed Issue is valid and can be reproduced labels Nov 26, 2017
@dastgirp
Copy link
Member

Related PR: #1893
@4144

@dastgirp
Copy link
Member

dastgirp commented Nov 26, 2017

Suggested fix:
npc_refresh

if (map->list[nd->bl.m].users) {

To

if (map->list[nd->bl.m].users && nd->class_ != FAKE_NPC) {

@4144
Copy link
Contributor

4144 commented Nov 26, 2017

@dastgir if this fixed issue, can you create pr?

@bWolfie
Copy link
Contributor

bWolfie commented Nov 28, 2017

how come fake_npc has sprite of female novice to begin with @_@
what i mean is how come female novice not something else :P

Also I tested that @dastgir no luck sorry

@MishimaHaruna
Copy link
Member

Suggested fix:

diff --git i/src/map/npc.c w/src/map/npc.c
index 10ff4ebf2..9e1414f9c 100644
--- i/src/map/npc.c
+++ w/src/map/npc.c
@@ -2708,7 +2708,7 @@ struct npc_data *npc_create_npc(enum npc_subtype subtype, int m, int x, int y, u
 	nd->area_size = AREA_SIZE + 1;
 	nd->class_ = class_;
 	nd->speed = 200;
-	nd->vd.class = 0;
+	nd->vd.class = INVISIBLE_CLASS;
 
 	return nd;
 }

4144 added a commit to 4144/Hercules that referenced this issue Nov 28, 2017
@ghost ghost removed the status:confirmed Issue is valid and can be reproduced label Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:core Affecting the Hercules core (i.e. not the game mechanics directly) severity:3-medium Issues with GM commands, script commands, and other commonly used features
Projects
None yet
Development

No branches or pull requests

5 participants