Increase Harvest on facets other than Felucca?

[Shazzy]

I want to have the harvest on other facets increased.
I see where they have it in Harvest Definitions and also in the Mining, Lumberjack scripts, but for some reason it is not working.

private int m_ConsumedPerHarvest, m_ConsumedPerFeluccaHarvest, m_ConsumedIlshenarHarvest, m_ConsumedPerTerMurHarvest;

and

public int ConsumedPerFeluccaHarvest
{
get
{
return this.m_ConsumedPerFeluccaHarvest;
}
set
{
this.m_ConsumedPerFeluccaHarvest = value;
}
}

public int ConsumedPerIlshenarHarvest
{
get
{
return this.m_ConsumedPerIlshenarHarvest ;
}
set
{
this.m_ConsumedPerIlshenarHarvest = value;
}
}

public int ConsumedPerTerMurHarvest
{
get
{
return this.m_ConsumedPerTerMurHarvest ;
}
set
{
this.m_ConsumedPerTerMurHarvest = value;
}
}

I have done the additions to Harvet Definition and also Harvest System.
But it is not working for Ilshenar of TerMur.
I have also added it into the mining and lumberjacking.

What am I missing on this? There has to be another call in this clunky mess, but I know I am old server I had double and triple resources on different facets.

Thanks for any assistance :]

Shazzy

 

[Dan(Tasanar)]

Need to somehow define what the IlshenarHarvest is. No where do I see Map.Ilshenar ect. Try searching for map.Felucca in those scripts.

 

[Visam]

I think what your looking for is the following:

starting at line 169 in Harvestsystem.cs

//The whole harvest system is kludgy and I'm sure this is just adding to it.
                        if (item.Stackable)
                        {
                            int amount = def.ConsumedPerHarvest;
                            int feluccaAmount = def.ConsumedPerFeluccaHarvest;

                            int racialAmount = (int)Math.Ceiling(amount * 1.1);
                            int feluccaRacialAmount = (int)Math.Ceiling(feluccaAmount * 1.1);

                            bool eligableForRacialBonus = (def.RaceBonus && from.Race == Race.Human);
                            bool inFelucca = (map == Map.Felucca);

                            if (eligableForRacialBonus && inFelucca && bank.Current >= feluccaRacialAmount && 0.1 > Utility.RandomDouble())
                                item.Amount = feluccaRacialAmount;
                            else if (inFelucca && bank.Current >= feluccaAmount)
                                item.Amount = feluccaAmount;
                            else if (eligableForRacialBonus && bank.Current >= racialAmount && 0.1 > Utility.RandomDouble())
                                item.Amount = racialAmount;
                            else
                                item.Amount = amount;

                            // Void Pool Rewards
                            item.Amount += WoodsmansTalisman.CheckHarvest(from, type, this);
                        }

 

[Shazzy]

I did put that part in and made no difference,

else
{
//The whole harvest system is kludgy and I'm sure this is just adding to it.
if (item.Stackable)
{
int amount = def.ConsumedPerHarvest;
int feluccaAmount = def.ConsumedPerFeluccaHarvest;
//int ilshenarAmount = def.ConsumedPerIlshenarHarvest;
//int termurAmount = def.ConsumedPerTerMurHarvest;

int racialAmount = (int)Math.Ceiling(amount * 1.1);
int feluccaRacialAmount = (int)Math.Ceiling(feluccaAmount * 1.1);

bool eligableForRacialBonus = (def.RaceBonus && from.Race == Race.Human);
bool inFelucca = (map == Map.Felucca);

if (eligableForRacialBonus && inFelucca && bank.Current >= feluccaRacialAmount && 0.1 > Utility.RandomDouble())
item.Amount = feluccaRacialAmount;
else if (inFelucca && bank.Current >= feluccaAmount)
item.Amount = feluccaAmount;
/*else if (inIlshenar && bank.Current >= ilshenarAmount)
item.Amount = ilshenarAmount;
else if (inTerMur && bank.Current >= termurAmount)
item.Amount = termurAmount;*/
else if (eligableForRacialBonus && bank.Current >= racialAmount && 0.1 > Utility.RandomDouble())
item.Amount = racialAmount;
else
item.Amount = amount;

// Void Pool Rewards
item.Amount += WoodsmansTalisman.CheckHarvest(from, type, this);
}

I had to mark them out because it did not recognize ilshenarAmount of termurAmount


[doublepost=1473951560][/doublepost]Ok All good. For anyone who wants this:

 else
 {
 //The whole harvest system is kludgy and I'm sure this is just adding to it.
 if (item.Stackable)
 {
 int amount = def.ConsumedPerHarvest;
 int feluccaAmount = def.ConsumedPerFeluccaHarvest;
 int ilshenarAmount = def.ConsumedPerIlshenarHarvest;
 int termurAmount = def.ConsumedPerTerMurHarvest;

 int racialAmount = (int)Math.Ceiling(amount * 1.1);
 int feluccaRacialAmount = (int)Math.Ceiling(feluccaAmount * 1.1);

 bool eligableForRacialBonus = (def.RaceBonus && from.Race == Race.Human);
 bool inFelucca = (map == Map.Felucca);
 bool inIlshenar = (map == Map.Ilshenar);
 bool inTerMur = (map == Map.TerMur);

 if (eligableForRacialBonus && inFelucca && bank.Current >= feluccaRacialAmount && 0.1 > Utility.RandomDouble())
 item.Amount = feluccaRacialAmount;
 else if (inFelucca && bank.Current >= feluccaAmount)
 item.Amount = feluccaAmount;
 else if (inIlshenar && bank.Current >= ilshenarAmount)
 item.Amount = ilshenarAmount;
 else if (inTerMur && bank.Current >= termurAmount)
 item.Amount = termurAmount;
 else if (eligableForRacialBonus && bank.Current >= racialAmount && 0.1 > Utility.RandomDouble())
 item.Amount = racialAmount;
 else
 item.Amount = amount;

 // Void Pool Rewards
 item.Amount += WoodsmansTalisman.CheckHarvest(from, type, this);
 }

I personally will add the racial bonus lines into the Ilshenar and terMur parts :]

Thanks!