Skip to content

Conversation

@LannyE
Copy link

@LannyE LannyE commented Jan 26, 2021

This is a large overhaul of the Solocraft module.

Features and Admin Controls added:

-Groups now work. Difficulty is now distributed across group size
-Debuff added for groups trying to exceed the difficulty modifier
-Debuff can be enabled and disabled
-Spellpower is now scaled based on character level and modifier
-Stat modifier can now be changed in config to reel in melee damage
-Individual modification of each dungeon/raid difficulty
-Max level cap to keep high level characters from getting buffed
-Max level cap modifiable in config for each dungeon/raid
-Dungeon/Raid types still used as a catch all if not found as an
individual instance in the config

A database table is included to manage groups and the difficulty
modifier that each group member has applied to them. It also
deals with server restarts and players having to log out and log
back in to try to correct stats. The table is required.

Updated README.md file with current instructions and config file name fix.

Fixes #9
closes #3

Testing:

Debuff:
-Character in dungeon with full difficulty modifier applied while not in a group.
-Invite another character to the group.
-Have new group member enter same dungeon.
-Debuff will be applied because of the group member already in the dungeon
has the full difficulty modifier applied.
-Have both/all characters leave dungeon and re-enter it to have the modifier
balanced across the current members of the group.

-Max Level:
-Solocraft.Max.Level.Diff and the Dungeon individual level are added together
to declare the max level a character can be to get the difficulty modifier
applied for the dungeon.
-Use a character whose level is higher than the max level and enter dungeon.
-Character should not receive the difficulty modifier buff for the dungeon.

-Spellpower and Stats modifier adjustments:
-Change modifier in the config.
-Enter dungeon with character and notice stats and/or spellpower numbers
changed from previous entrance into the dungeon.

This is a large overhaul of the Solocraft module.

Features and Admin Controls added:

-Groups now work. Difficulty is now distributed across group size
-Debuff added for groups trying to exceed the difficulty modifier
-Debuff can be enabled and disabled
-Spellpower is now scaled based on character level and modifier
-Stat modifier can now be changed in config to reel in melee damage
-Individual modification of each dungeon/raid difficulty
-Max level cap to keep high level characters from getting buffed
-Max level cap modifiable in config for each dungeon/raid
-Dungeon/Raid types still used as a catch all if not found as an
individual instance in the config

A database table is included to manage groups and the difficulty
modifier that each group member has applied to them. It also
deals with server restarts and players having to log out and log
back in to try to correct stats. The table is required.

Updated README.md file with current instructions and config file name fix.

Testing:

Debuff:
-Character in dungeon with full difficulty modifier applied while not in a group.
-Invite another character to the group.
-Have new group member enter same dungeon.
-Debuff will be applied because of the group member already in the dungeon
has the full difficulty modifier applied.
-Have both characters leave dungeon and re-enter it to have the modifier
balanced across the current members of the group.

-Max Level:
-Solocraft.Max.Level.Diff and the Dungeon individual level are added together
to declare the max level a character can be to get the difficulty modifier
applied for the dungeon.
-Use a character whose level is higher than the max level and enter dungeon.
-Character should not receive the difficulty modifier buff for the dungeon.

-Spellpower and Stats modifier adjustments:
-Change modifier in the config.
-Enter dungeon with character and notice stats and/or spellpower numbers
changed from previous entrance into the dungeon.
Removed: unused variable maxlevel at line 508
@Helias
Copy link
Member

Helias commented Jan 26, 2021

Now this has conflicts with the other PR that I just merged 😅
Solve conflicts please

@LannyE
Copy link
Author

LannyE commented Jan 26, 2021

This is an adventure. You can definitely tell I am new to this. Hopefully they have been resolved correctly.

LannyE and others added 3 commits January 26, 2021 13:51
Large nested If groups removed and converted into using Hashmaps
@LannyE
Copy link
Author

LannyE commented Jan 28, 2021

Thanks for showing me the hashmap option. That greatly reduced the number of lines of code. I made the changes and did some testing. It seems to be working fine.

@Helias
Copy link
Member

Helias commented Jan 28, 2021

It is much better, doesn't it? 😁
Good job!

@Helias
Copy link
Member

Helias commented Jan 28, 2021

I improved the comments next-to the dungeons.
Anyways, a hints for the future, convert tabs into spaces 😉

I will wait the build and merge it if the bild succeed.

@LannyE
Copy link
Author

LannyE commented Jan 28, 2021

The changes looks good. Thank you for all your help. I appreciate your patience while I navigate this learning curve. I haven't programmed in C++ for about 20+ years and never used Git (which is obvious). On a side note: I also have a bunch of database changes for the lunar event and a new module I made based off a script I found. Once I get some free time I will take a shot at uploading those.

@LannyE
Copy link
Author

LannyE commented Jan 28, 2021

Also this push should fix issue #3. I used the testing done in that issue to form the logic and config settings.

@Helias
Copy link
Member

Helias commented Jan 28, 2021

About #3 I edited the description of the PR so when we merge the PR the issue will be closed, for the module and lunar make a PR and create a repo with your module ;-)

@Helias Helias merged commit 4678c78 into azerothcore:master Jan 28, 2021
@LannyE LannyE deleted the Solocraft-Overhaul branch February 23, 2022 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logging out while in a dungeon and logging back in does not rebuff the character More Condtions Needed for Different Dungeon/Raid Type Scaling

2 participants