Couple of Fixes

[xwerswoodx]

• Fixed: The issue that causes more1/2 not saved correctly for spellbook. (Issue: Error setting more1/more2 Value on a spellbook (may affect other variable assignation) #1221)

• Fixed: Players automatically become criminal to everyone when attack someone. (Issue: Criminal & Guard Problem. #1213)

• Fixed: CallGuard not working correctly on the players that only criminal for target. (Issue: Criminal & Guard Problem. #1213, CChar::CallGuards() function #1040)

• Fixed: The wrong input of Kill trigger, to make it compatible with older sphere versions. (Issue: @KILL -> SRC should be killer, not victim #1210)

• Added: New Layer LAYER_STORAGE (80) to let scripters create their own storage system. (Issue: Discussion: Server layers system #1209)
  Only t_container and t_container_locked can be equipped to this layer.

• Changed: Decay time removed from spawned items, as Sphere gives tons of invalid link errors when items decayed. (Issue: t_spawn_item #1218)

• Added: New spellflag SPELLFLAG_ASYNC to randomize field timers like old sphere versions. (Issue: [Request] Fields diff time removed #1169)

• Added: New spellflag SPELLFLAG_TARG_ONLYSELF to make spell only be used on character himself. (Issue: [Feature Request] SPELLFLAG_ONLYSELF #1171)

• Changed: SERV.LOG command changed to support console colors. (Issue: [Feature Request] Colored Serv & Src Log System #1158)

SERV.LOG works same as before.
Usage of custom color system is: SERV.LOG @color,type,mask message
Example: SERV.LOG @CTCOL_RED,LOGL_EVENT,LOGM_NOCONTEXT|LOGM_KILLS message (If CTCOLs, LOGLs and LOGMs added to Defs)

• Added: Sphere.ini setting CanPetsDrinkPotion (True in default) to make pets drink potion when their owner drop it on it. (Issue: [Feature Request] NPCs can drink potion. #1147)
  CanPetsDrinkPotion = 0 can be added to sphere.ini to disable that.

• Fixed: Missing fCheckOnly checks block the taming skill. (Issue: Taming and OF_PetSlots bug? #1219)

• Added: New trigger @drink added for characters. (Issue: [Feature Request] New trigger: @Drink #1162)

• Added: New trigger @AFKmode added for characters. (Issue: [Feature Request] New trigger: @AfkMode #1161)

• Added: New trigger @reveal added for characters. (Issue: [Feature Request] New trigger: @Reveal #1157)

• Added: New triggers @ArrowQuest_Add and @ArrowQuest_Close added for characters. (Issue: m_pclient #1116)

• Added: New function triggers f_onaccount_block and f_onaccount_unblock. (Issue: [Feature Request] f_onaccount_block and f_onaccount_unblock #1156)

• Added: New function trigger f_onserver_broadcast. (Issue: [Feature Request] f_onserver_broadcast #1145)

• Added: Two new LOCAL variables ItemPoisonReductionChance and ItemPoisonReductionAmount to @hit trigger. (Issue: Fixed: Ship plank not remaining open when turning the ship around. In addition the ship plank will autoclose after 5 seconds. ( Issue #1089) #1102

• Added: You can now use TDATA2 for setting default cooldown (delay) for drinks. (Default: 15secs for potions, 1500 secs for ales) (Issue: [Feature Request] New trigger: @Drink #1162 (comment))

• Changed: Reactive armor range increased from 1 to 2 in default.

• Fixed: Possible fix for accesses and bans not loading for multis. (Issue: House accesslist / banlist #1223)

• Fixed: Possible fix for wrong damage calculation of slayer bonus damage. (Issue: Slayer property works on NPCs with faction 00 or different #1179)

• Added: REGION_FLAG_HOUSE (0x0100) for houses. (Issue: [Feature Request] REGION_FLAG_SHIP like REGION_FLAG_HOUSE #1154)

• Added: ARGO object for @Murdermark trigger to get killed character.

• Fixed: ARGN1 (reap amount) doesn't updated depends on the amount left on bit. (Issue: @RegionResourceGather & @ResourceGather - Small bug with ARGN1 #1224)

• Added: ARGO to get bit uid under @ResourceGather and @RegionResourceGather triggers.

• Fixed: Possible fix for the OnRegionTrigger getting called on invalid region while multi creating/removing. (Issue: https://discord.com/channels/354358315373035542/355031741569826816/1223978839336554627)

• Changed: dword CAN flags changed with uint64 to increase limits.

• Fixed: Possible fix for chat debugs. (Issue: Chat Debug #1212) (Issue: https://discord.com/channels/354358315373035542/483721253773901824/1224030729906815077)

• Added: "MOREX" key to set amount of fruit given when the crop ripped.

• Changed: Now @create trigger called when crop grow to let people to change MORE values in stages if needed.

• Added: @ResourceTest trigger added for crops. (This trigger works separately for every stage)

• Added: @ResourceGather trigger added for crops. (This trigger works separately for every stage)

• Fixed: Possible fix for invalid ResourceID warnings. (Issue: Console warning: t_rock | t_crops | t_meat_raw | t_food #1071)

• Fixed: Missing REGION reference for multis. (Issue: region.tag (t_multi) #1228)

• Added: SPELLFLAG_NO_ELEMENTALENGINE (0400000000) as spellflag to ignore default elemental engine behaviours for specific spells. (Issue: Spellbook Spells behaviour (OLD vs OSI) (Feature Request) #1230)

• Fixed: An issue that causes T_MULTI_CUSTOM keep multi count saved. (Issue: t_multi_custom remove #1232)

• Fixed: Bonded pets cannot see their master while DeadCannotSeeLiving equals to 2. (Issue: Pets with property "Bonded" not following after death #1225)

• Fixed: The issue that causes Guilds without align shown enemy to the players with a guild with align. (Issue: Fix guild notoriety issues #1208)

• Fixed: GMs cannot dress up players. (Issue: Impossible to drag and drop equip item on other player. #901)

• Fixed: Wrong check block GMs to unequip item from players. (Impossible to drag and drop equip item on other player. #901)

• Fixed: An issue that causes a visual bug to remove all tiles while removing a tile in higher floors. (Issue: custom housing (the same bug as issue #323) #345)

• Merged: Jhobean's EMOTEF_EAT pr merged. (Issue: Add ini switch for removing emote when eating. Now send object pointer to Eat trigger  #1207)

• Changed: EMOTEF_EAT name changed to EMOTEF_HIDE_EAT_PLAYER and limited to only player emotes.

• Added: EMOTEF_HIDE_EAT_NPC to hide emotes while eating only for npcs.

• Added: Trigger @hit and @GetHit are added as an item trigger.

• Added: MAPVIEWRADAR, MAPVIEWSIZE and MAPVIEWSIZEMAX to sphere.ini that overrides default visual range values. (Issue: [Feature Request] Visual Range diff set sphere.ini settings #1144)

• Added: 'H' shortcut for variables to get the value as hexadecimal.

• Merged: raydienull's LAYER_STABLE build merged, but LAYER_AUCTION keep same as LAYER_STABLE added as a new layer. (PR: Change LAYER_AUCTION to LAYER_STABLE and now is used to store the sta… #1201)

• Fixed: raydienull's LAYER_STABLE follower triggers doesn't called while taking pet out of stable.

• Fixed: raydienull's LAYER_STABLE doesn't work with the new follower system.

• Removed: Unnecessary LINK == PLAYER checks as figurines will be stocked on the owned player.

• Fixed: Setting GMs dispid to c_man_gm in-game, prevents GMs to use in-game commands.

• Changed: Default NPC hear range reverted back to the 14 from 4, as it was 14 in older versions, as there was no reason for changing it. (Issue: Follower Commands Don't Work. #1215)

• Rework: Champion system fully reworked.

• Fixed: Candles keep stuck while killing npcs. (Issue: HardCoded Champion System, Stuck candle #564)

• Fixed: LEVELMAX doesn't affect the max level of champion system, and keep spawn npcs for default max level.

• Changed: Spawn system changed and now it automatically calculate the percentage of spawns per level in descending order.

• Changed: Candle calculation is now depend on max level, instead of static 4 level.

• Added: SRC added in @start trigger. SRC is a reference of player that starts champion spawner.

• Added: @complete (Issue: Hardcoded Champion System - Enhancement and ideas. #477), @DelRedCandle, @DelWhiteCandle, @Level and @Stop triggers to champion system.

• Fixed: Script multis move 1 block north when their type set as t_multi_addon. (Issue: T_Multi_Addons Position #1238)

• Fixed: ADDCOMPONENT command was added as ADDCOMP for houses. (Issue: ADDCOMP, DELCOMP #1239)

• Changed: Re-activated 56b style hardcoded fishing system, as I don't know why it's deactivated as softcoded system too bad and broken.

• Added: EF_DAMAGETOOLS flag implemented to the sphere. (Only affects the skills with SKF_GATHER flag.)

• Added LOCAL.ITEMDAMAGECHANCE to the @success trigger for skills to decide damage chance for tools. (Default: 25)

• Added LOCAL.ITEMDAMAGEAMOUNT to the @success trigger for skills to decide damage amount. (Default: 1)

[GladieUO]

Just tried the latest version of PR and most of my system are not working. There is something different with tags returning 00 instead of 0 and same with local.x returning 01 if it should be 1. 😲 Is this intended?

[xwerswoodx]

Just tried the latest version of PR and most of my system are not working. There is something different with tags returning 00 instead of 0 and same with local.x returning 01 if it should be 1. 😲 Is this intended?

Yes, they should return in hex like older versions, it was a mistake that we changed it to decimal, it wasn't intentionally, so you can use dTAG to get them decimal like older versions. We discussed it on here;

#1222 (comment)

[Jhobean]

Yes, they should return in hex like older versions, it was a mistake that we changed it to decimal, it wasn't intentionally, so you can use dTAG to get them decimal like older versions. We discussed it on here;

I liked the new X behavior... No really agree to revert back...

[xwerswoodx]

I liked the new X behavior... No really agree to revert back...

It's understandable, we can discuss about it, honestly using different system between versions, causes many script-side issues, so I agree it cause many issues when we change it in middle of development, but honestly I can't decide this by myself.

[Jhobean]

Since day one i'm on X I tought it was a new feature and changed all my script. Impact is big on dialog and when you see visual value or when you compare two data.
I talked about thia 4 years ago and it was say it was like this. I appreciate all work you do when bug is fixed but this change make frustrating huge difference when coding shard.

[GladieUO]

Im curious how it was on 56b and 56c since I dont remember we ever needed to use anything else than just simple check for tag.xxx returning decimals. Tbh taking some functionalities from the old spheres is always iffy, since we all know they were all over the place.

If in the X its working for years some way, mistake or not I think everyone using X is now used to it and cant imagine anyone appreciating this change. For servers in process, not launched, OK extra annoying work to change everything and check that it works again. But for someone already running live shard and wanting to keep with the updates, this can break a lot of stuff.

Whatever the decision will be, i guess we adapt to it and still appreciate rest of your work. 👍

Additionally I guess we can always revert this change for ourselves and recompile everytime there is new update.

[cbnolok]

Also i'd like to discuss issue #1171 since it's addressed in this pr

[cbnolok]

Im curious how it was on 56b and 56c since I dont remember we ever needed to use anything else than just simple check for tag.xxx returning decimals. Tbh taking some functionalities from the old spheres is always iffy, since we all know they were all over the place.

If in the X its working for years some way, mistake or not I think everyone using X is now used to it and cant imagine anyone appreciating this change. For servers in process, not launched, OK extra annoying work to change everything and check that it works again. But for someone already running live shard and wanting to keep with the updates, this can break a lot of stuff.

Whatever the decision will be, i guess we adapt to it and still appreciate rest of your work. 👍

Additionally I guess we can always revert this change for ourselves and recompile everytime there is new update.

Really does that change break so much stuff? I'm quite sure that on 56d and before everything returned hex values. That was the reason for implementing in the past dtag, dlocal etc. I don't remember when i changed that part of code, but i can say that change wasn't from the start of the fork.
I see the benefits of both returning hex and decimal, i'd say to keep the historical behavior but i'd like to understand how this revert would actually impact shards. Obviously some changes are needed to be done on scripts, but really that much?

[cbnolok]

Once we sort out this and the new targ spellflag, i think this pr is ready for being merged

[xwerswoodx]

Once we sort out this and the new targ spellflag, i think this pr is ready for being merged

I revert back the spellflag update as SPELLFLAG_FX_TARG flag enough for event, I tested it and seems like effects works with spellflag_fx flags instead of spellflag_targ flags.

For decimal/hexadecimal return, we were discussing it in discord channel.

[GladieUO]

Really does that change break so much stuff? I'm quite sure that on 56d and before everything returned hex values. That was the reason for implementing in the past dtag, dlocal etc. I don't remember when i changed that part of code, but i can say that change wasn't from the start of the fork. I see the benefits of both returning hex and decimal, i'd say to keep the historical behavior but i'd like to understand how this revert would actually impact shards. Obviously some changes are needed to be done on scripts, but really that much?

From the first quick look, it broken all the dialogs using any kind of tags or locals. Which in general is telling me, that I will need to go through every single tag and local condition, test ingame if its work, and if not add one letter to make it work. For me personaly that means go through everything ive been working on for year and half and marked as done and tested, test and check again.
I personaly dont see any benefits from using 16 as default, but im just lil pup to understand the purpose.

As I said, thats just me, if the change is needed or should have been done long time ago, we shall adapt.

[xwerswoodx]

Really does that change break so much stuff? I'm quite sure that on 56d and before everything returned hex values. That was the reason for implementing in the past dtag, dlocal etc. I don't remember when i changed that part of code, but i can say that change wasn't from the start of the fork. I see the benefits of both returning hex and decimal, i'd say to keep the historical behavior but i'd like to understand how this revert would actually impact shards. Obviously some changes are needed to be done on scripts, but really that much?

From the first quick look, it broken all the dialogs using any kind of tags or locals. Which in general is telling me, that I will need to go through every single tag and local condition, test ingame if its work, and if not add one letter to make it work. For me personaly that means go through everything ive been working on for year and half and marked as done and tested, test and check again. I personaly dont see any benefits from using 16 as default, but im just lil pup to understand the purpose.

As I said, thats just me, if the change is needed or should have been done long time ago, we shall adapt.

We decided to add h keyword (hTAG, hLOCAL, etc.) and an ini switch for default return, you can add DecimalVariables=1 in sphere.ini to make sure TAGs and LOCALs return as decimal instead of hexadecimal.

Done