Guild command - Dues Error

A place to submit .patch fixes for the DOL SVN

Moderator: Developer Team

Guild command - Dues Error

Postby Crazys » Fri Oct 03, 2014 3:07 pm

Issue - Old dues breaks from this amount line existing and overriding the 2 value.

Remove Line 2247
client.Player.Guild.SetGuildDuesPercent(amount);
Crazys
Contributor
 
Posts: 346
Joined: Tue Nov 07, 2006 10:18 pm

Re: Guild command - Dues Error

Postby Leodagan » Fri Oct 03, 2014 3:24 pm

Crazys, can you provide the file in which this is found ? :D

Which 2 values are being overrided ? are they used in different code part ?
User avatar
Leodagan
Developer
 
Posts: 1350
Joined: Tue May 01, 2012 9:30 am
Website: https://daoc.freyad.net
Location: Lyon

Re: Guild command - Dues Error

Postby Crazys » Fri Oct 03, 2014 3:35 pm

Crazys, can you provide the file in which this is found ? :D

Which 2 values are being overrided ? are they used in different code part ?

before
Sorry!!! guild.cs (in GameServer/Commands/Player)
Code: Select all
else if (amount > 0 && amount <= 100) { client.Player.Guild.SetGuildDues(true); if (ServerProperties.Properties.NEW_GUILD_DUES) { client.Player.Guild.SetGuildDuesPercent(amount); client.Out.SendMessage(LanguageMgr.GetTranslation(client.Account.Language, "Scripts.Player.Guild.DuesOn", amount), eChatType.CT_Guild, eChatLoc.CL_SystemWindow); } else { client.Player.Guild.SetGuildDuesPercent(2); client.Out.SendMessage(LanguageMgr.GetTranslation(client.Account.Language, "Scripts.Player.Guild.DuesOn", 2), eChatType.CT_Guild, eChatLoc.CL_SystemWindow); } client.Player.Guild.SetGuildDuesPercent(amount); }
After
Code: Select all
else if (amount > 0 && amount <= 100) { client.Player.Guild.SetGuildDues(true); if (ServerProperties.Properties.NEW_GUILD_DUES) { client.Player.Guild.SetGuildDuesPercent(amount); client.Out.SendMessage(LanguageMgr.GetTranslation(client.Account.Language, "Scripts.Player.Guild.DuesOn", amount), eChatType.CT_Guild, eChatLoc.CL_SystemWindow); } else { client.Player.Guild.SetGuildDuesPercent(2); client.Out.SendMessage(LanguageMgr.GetTranslation(client.Account.Language, "Scripts.Player.Guild.DuesOn", 2), eChatType.CT_Guild, eChatLoc.CL_SystemWindow); } }
Just removing the 1 line. Its overriding the else statement if New_Guild_Dues is false.
Not home currently to be able to make a patch.
Crazys
Contributor
 
Posts: 346
Joined: Tue Nov 07, 2006 10:18 pm

Re: Guild command - Dues Error

Postby Leodagan » Fri Oct 03, 2014 4:24 pm

Ok that's clearly an error to me ;)

You should manage some way to enable your Source Forge Account for HTTPS SVN access, I know I had some trouble with it recently, I fallback to SVN+SSH (which is pretty simple if you're a Unix user...) but I haven't tried recently if HTTPS is working again :)

I can try to commit that quickly.

edit - done : 3367
User avatar
Leodagan
Developer
 
Posts: 1350
Joined: Tue May 01, 2012 9:30 am
Website: https://daoc.freyad.net
Location: Lyon


Return to “%s” DOL Code Contributions

Who is online

Users browsing this forum: No registered users and 1 guest