-
Notifications
You must be signed in to change notification settings - Fork 417
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
base: master
Are you sure you want to change the base?
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.
common/evolving.cpp
Outdated
|
||
void EvolvingItemsManager::LoadEvolvingItems() const | ||
{ | ||
auto const &results = ItemsEvolvingDetailsRepository::All(*m_db); |
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 this supposed to use the m_content_db
?
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.
Maybe. I made the assumption that it would move to a content table and be static based on in era items. If that is the decision taken, and we get a content table made and populated, I will update and re-test accordingly. For my testing I used a local table.
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! |
Add logging and player events
Updates to resolve review comments and a rebase.
migrate to content_db for items_evolving_details
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. |
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