-
Notifications
You must be signed in to change notification settings - Fork 427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Evolving Item Support for RoF2 #4496
Conversation
Can't wait to try this out :D |
So Excited, Neckkola dropping the big features! |
@@ -212,6 +214,7 @@ namespace DatabaseSchema { | |||
"ground_spawns", | |||
"horses", | |||
"items", | |||
"items_evolving_details", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the sql for this table missing from the update manifest?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is missing at the moment as it will likely be a content table, and will be loaded that way.
Hi. Can't wait to try this. Will evolved items work on bots? Or is it limited to clients only? |
f569ba3
to
82b1b36
Compare
Updated based on review comments. Going to run through some tests |
How'd testing go. Are you ready for merge? |
Were you able to test this on bots? Curious if they'll evolve while equipped. ty! |
82b1b36
to
ffc964b
Compare
Good afternoon Update the items_evolving_details table to the content_db, and provided a sql in utils to populate. It only has two items with evolving details though all items flagged as evolving are present. As more detail is known, it can be easily updated and activated. Some cleanup as well. I need to create the documentation next. I believe that the other comments are complete. |
Documentation has been updated. |
Looking to get this one merged in this week |
ffc964b
to
d84918d
Compare
Add logging and player events
Updates to resolve review comments and a rebase.
migrate to content_db for items_evolving_details
e11949e
to
f3b4e48
Compare
Fix empty faction search results
Description
Provides evolving item support for the RoF2 Client. This is draft only at this time.
What it requires:

At the moment, a content table is required for the current design. A table 'items_evolving_details' is required with the following sql
This table contains the evolving criteria for each item as described here:
Details are in namespace EvolvingItems::Types and SubTypes.
Logic is in Client::ProcessEvolvingItem
Here is my test table with two items:

What it does:
Outstanding:
Perl
$item->AddEvolveAmount(amount)
.$item->GetAugmentEvolveUniqueID(slot_id)
.$item->GetEvolveActivated()
.$item->GetEvolveAmount()
.$item->GetEvolveCharacterID()
.$item->GetEvolveEquipped()
.$item->GetEvolveFinalItemID()
.$item->GetEvolveItemID()
.$item->GetEvolveLevel()
.$item->GetEvolveLoreID()
.$item->GetEvolveProgression()
.$item->GetEvolveUniqueID()
.$item->IsEvolving()
.$item->SetEvolveAmount(amount)
.$item->SetEvolveProgression(amount)
.Perl Example
Lua
item:AddEvolveAmount(amount)
.item:GetAugmentEvolveUniqueID(slot_id)
.item:GetEvolveActivated()
.item:GetEvolveAmount()
.item:GetEvolveCharacterID()
.item:GetEvolveEquipped()
.item:GetEvolveFinalItemID()
.item:GetEvolveItemID()
.item:GetEvolveLevel()
.item:GetEvolveLoreID()
.item:GetEvolveProgression()
.item:GetEvolveUniqueID()
.item:IsEvolving()
.item:SetEvolveAmount(amount)
.item:SetEvolveProgression(amount)
.Lua Example
Type of change
Testing
I have tested with RoF2 in various forms. Would appreciate others' testing results.
Clients tested:
RoF2
Checklist