Allowing (other job) Soul Links to be casted on yourself

[Reinheit]

Since diffing Secret's Soul Link modification (which is awesome af, outside of the memory leak I might add)... I've been experimenting with different SL balance changes. One of those involves being able to cast another job's Soul Link on myself --Soul Linker-- as well. Unsuccessfully so far, I might add.

I'm hoping the fine folks here would be able to guide me to the right solution, as it seems my very limited understanding of script command solutions don't work in src. I sincerely apologize in advance for the eyes this might hurt -- just trying to learn 

In trying to get the Monk Soul Link to work on myself, here's what I've tried so far -- with the results that followed:

• I'd removed the NoTargetSelf flag from skill_db.yml --> can cast on myself, but no SL effect is actually enabled. 

Using these lines in status.cpp: 

/* Storing the target job rather than simply SC_SPIRIT simplifies code later on */
	SkillStatusChangeTable[skill_get_index(SL_ALCHEMIST)]	= (sc_type)MAPID_ALCHEMIST,
	SkillStatusChangeTable[skill_get_index(SL_MONK)]	= (sc_type)MAPID_MONK,
	SkillStatusChangeTable[skill_get_index(SL_STAR)]	= (sc_type)MAPID_STAR_GLADIATOR,
	SkillStatusChangeTable[skill_get_index(SL_SAGE)]	= (sc_type)MAPID_SAGE,
	SkillStatusChangeTable[skill_get_index(SL_CRUSADER)]	= (sc_type)MAPID_CRUSADER,
	SkillStatusChangeTable[skill_get_index(SL_SUPERNOVICE)]	= (sc_type)MAPID_SUPER_NOVICE,
	SkillStatusChangeTable[skill_get_index(SL_KNIGHT)]	= (sc_type)MAPID_KNIGHT,
	SkillStatusChangeTable[skill_get_index(SL_WIZARD)]	= (sc_type)MAPID_WIZARD,
	SkillStatusChangeTable[skill_get_index(SL_PRIEST)]	= (sc_type)MAPID_PRIEST,
	SkillStatusChangeTable[skill_get_index(SL_BARDDANCER)]	= (sc_type)MAPID_BARDDANCER,
	SkillStatusChangeTable[skill_get_index(SL_ROGUE)]	= (sc_type)MAPID_ROGUE,
	SkillStatusChangeTable[skill_get_index(SL_ASSASIN)]	= (sc_type)MAPID_ASSASSIN,
	SkillStatusChangeTable[skill_get_index(SL_BLACKSMITH)]	= (sc_type)MAPID_BLACKSMITH,
	SkillStatusChangeTable[skill_get_index(SL_HUNTER)]	= (sc_type)MAPID_HUNTER,
	SkillStatusChangeTable[skill_get_index(SL_SOULLINKER)]	= (sc_type)MAPID_SOUL_LINKER,

___

// Replaced the end of this line:
	SkillStatusChangeTable[skill_get_index(SL_MONK)]		= (sc_type)MAPID_MONK,

// with this:
	SkillStatusChangeTable[skill_get_index(SL_MONK)]		= (sc_type)MAPID_SOUL_LINKER,

• This allowed me to cast on myself! But obv it replaced the ability to cast on monks. 

___

// Added a duplicate line with a different MAPID_ below it:
SkillStatusChangeTable[skill_get_index(SL_MONK)]		= (sc_type)MAPID_MONK,
SkillStatusChangeTable[skill_get_index(SL_MONK)]		= (sc_type)MAPID_SOUL_LINKER,

• This allowed me to cast on myself, but removed the ability to cast on Monks

___

// tried using || as if (sc_type) were an expression >_< 
SkillStatusChangeTable[skill_get_index(SL_MONK)]		= (sc_type)MAPID_MONK || MAPID_SOUL_LINKER,

• This gave me an error when compiling (and obviously didn't work):
  • '=': cannot convert from 'bool' to 'sc_type'
  • Wasn't even able to load up map server

________________

Would anyone be able to advise? Thanks for taking the time to read ~ 

Edited February 9, 2021 by Reinheit