kyeme 71 Posted July 19, 2013 I suggest that we should have a config. in WOE that we can Invite/Expel players while inside the Guild Castle (because in the official server this is the Old settings - I'm not sure if it is the actual settings now.) But if it is the official behavior, I suggest we should have a config just like the Old one. e.g // <msg>// <msg>castle_member_invite: yescastle_member_expel: yes Quote Share this post Link to post Share on other sites
p!\/aT3 0 Posted July 20, 2013 yes this would be a big help. Quote Share this post Link to post Share on other sites
pan 87 Posted January 21, 2014 That's a good idea, and it wouldn't be difficult to implement either. I've changed the names of those config options. Open src/map/battle.c and find: { "case_sensitive_aegisnames", &battle_config.case_sensitive_aegisnames, 1, 0, 1, },Add below:{ "guild_castle_invite", &battle_config.guild_castle_invite, 0, 0, 1, },{ "guild_castle_expulsion", &battle_config.guild_castle_expulsion, 0, 0, 1, },Open src/map/battle.h and find:int case_sensitive_aegisnames;Add below: int guild_castle_invite; int guild_castle_expulsion;Finally open src/map/guild.c and find (@guild_invite): if( tsd->status.guild_id > 0 || tsd->guild_invite > 0 || ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle)Replace it with: if( tsd->status.guild_id > 0 || tsd->guild_invite > 0 || ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle && !battle_config.guild_castle_invite)Find (@guild_leave): if( sd->status.account_id != account_id || sd->status.char_id != char_id || sd->status.guild_id != guild_id || ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle)Replace it with: if( sd->status.account_id != account_id || sd->status.char_id != char_id || sd->status.guild_id != guild_id // Can't leave inside castles || ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle && !battle_config.guild_castle_expulsion)Find (@guild_expulsion): if ((tsd = map->id2sd(account_id)) && tsd->status.char_id == char_id && ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle)Replace it with: if ((tsd = map->id2sd(account_id)) && tsd->status.char_id == char_id && ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle && !battle_config.guild_castle_expulsion)In your conf/battle/guild.conf add:// Can guild members invite/expel members inside guild castles in WoE/GvG? (Note 1)// default noguild_castle_invite: noguild_castle_expulsion: no 3 Mystery, kyeme and jaBote reacted to this Quote Share this post Link to post Share on other sites
mkbu95 34 Posted January 21, 2014 +1 for this Quote Share this post Link to post Share on other sites
Ind 945 Posted February 10, 2014 I personally like it but the lack of support makes me wonder whether its worth to put it in the core, perhaps a plugin would do best? (I wouldn't mind coding the plugin myself) Quote Share this post Link to post Share on other sites
AnnieRuru 957 Posted February 10, 2014 (edited) invite/expel a guild member is part of the behavior of guildlock mapflag http://rathena.org/wiki/Mapflag#guildlock why not just add guildlock mapflag on these castle maps ? EDIT: ok I was wrong ... again the original behavior was already disallow to invite/expel now he wants a configuration that allows to invite/expel Edited February 17, 2014 by AnnieRuru Quote Share this post Link to post Share on other sites
Ind 945 Posted February 10, 2014 ohoho as i understand that kills the proposal, indeed Quote Share this post Link to post Share on other sites
kyeme 71 Posted February 16, 2014 That's a good idea, and it wouldn't be difficult to implement either. I've changed the names of those config options.Open src/map/battle.c and find: { "case_sensitive_aegisnames", &battle_config.case_sensitive_aegisnames, 1, 0, 1, },Add below:{ "guild_castle_invite", &battle_config.guild_castle_invite, 0, 0, 1, },{ "guild_castle_expulsion", &battle_config.guild_castle_expulsion, 0, 0, 1, },Open src/map/battle.h and find:int case_sensitive_aegisnames;Add below: int guild_castle_invite; int guild_castle_expulsion;Finally open src/map/guild.c and find (@guild_invite): if( tsd->status.guild_id > 0 || tsd->guild_invite > 0 || ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle)Replace it with: if( tsd->status.guild_id > 0 || tsd->guild_invite > 0 || ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle && !battle_config.guild_castle_invite)Find (@guild_leave): if( sd->status.account_id != account_id || sd->status.char_id != char_id || sd->status.guild_id != guild_id || ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle)Replace it with: if( sd->status.account_id != account_id || sd->status.char_id != char_id || sd->status.guild_id != guild_id // Can't leave inside castles || ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle && !battle_config.guild_castle_expulsion)Find (@guild_expulsion): if ((tsd = map->id2sd(account_id)) && tsd->status.char_id == char_id && ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle)Replace it with: if ((tsd = map->id2sd(account_id)) && tsd->status.char_id == char_id && ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle && !battle_config.guild_castle_expulsion)In your conf/battle/guild.conf add:// Can guild members invite/expel members inside guild castles in WoE/GvG? (Note 1)// default noguild_castle_invite: noguild_castle_expulsion: no Please add to the main repo. Thanks <3 Quote Share this post Link to post Share on other sites
kyeme 71 Posted May 20, 2014 That's a good idea, and it wouldn't be difficult to implement either. I've changed the names of those config options.Open src/map/battle.c and find: { "case_sensitive_aegisnames", &battle_config.case_sensitive_aegisnames, 1, 0, 1, },Add below:{ "guild_castle_invite", &battle_config.guild_castle_invite, 0, 0, 1, },{ "guild_castle_expulsion", &battle_config.guild_castle_expulsion, 0, 0, 1, },Open src/map/battle.h and find:int case_sensitive_aegisnames;Add below: int guild_castle_invite; int guild_castle_expulsion;Finally open src/map/guild.c and find (@guild_invite): if( tsd->status.guild_id > 0 || tsd->guild_invite > 0 || ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle)Replace it with: if( tsd->status.guild_id > 0 || tsd->guild_invite > 0 || ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle && !battle_config.guild_castle_invite)Find (@guild_leave): if( sd->status.account_id != account_id || sd->status.char_id != char_id || sd->status.guild_id != guild_id || ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle)Replace it with: if( sd->status.account_id != account_id || sd->status.char_id != char_id || sd->status.guild_id != guild_id // Can't leave inside castles || ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle && !battle_config.guild_castle_expulsion)Find (@guild_expulsion): if ((tsd = map->id2sd(account_id)) && tsd->status.char_id == char_id && ((map->agit_flag || map->agit2_flag) && map->list[tsd->bl.m].flag.gvg_castle)Replace it with: if ((tsd = map->id2sd(account_id)) && tsd->status.char_id == char_id && ((map->agit_flag || map->agit2_flag) && map->list[sd->bl.m].flag.gvg_castle && !battle_config.guild_castle_expulsion)In your conf/battle/guild.conf add:// Can guild members invite/expel members inside guild castles in WoE/GvG? (Note 1)// default noguild_castle_invite: noguild_castle_expulsion: no Up~ Quote Share this post Link to post Share on other sites
pan 87 Posted May 20, 2014 Should no objections or other points come forward within the next week I'll commit this change to our main repository. 1 kyeme reacted to this Quote Share this post Link to post Share on other sites
cJei 4 Posted May 22, 2014 NO objections! Thank you Quote Share this post Link to post Share on other sites
Mystery 594 Posted May 29, 2014 This has been added in: https://github.com/HerculesWS/Hercules/commit/407502abc921bb4cf06ff462ab6e2f23a327e950 1 cJei reacted to this Quote Share this post Link to post Share on other sites