Feature Use Preferences.resetToDefault( String ) in ValhallaManager.resetPerAscensionCounters()

fredg1

Member
We should take a habit of using Preferences.resetToDefault( String ), rather than manually setting every preference to a set value, in ValhallaManager.resetPerAscensionCounters()

I did the transition myself, and am ready to submit a patch, but while doing it, I found four discrepancies. Two of which I'm just not sure about, and two that are actual bugs.

Not sure:
  • nextParanormalActivity has a default value of 0, but is reset to 1 on ascension. Is that correct?
  • We're not resetting noobDeferredPoints on ascension. Is that... correct?
Bugs:
  • backupCameraMode has a default of <blank>, but should have meat as a default.
  • bondDR is mistakenly referred to as bondDA half of the time.

    Referred to as bondDA:

    In ValhallaManager:
    Java:
            Preferences.setBoolean( "bondDA", false );
    In ChoiceManager:
    Java:
            Preferences.setBoolean( "bondDA", bondDR );

    Referred to as bondDR:

    In defaults.txt:
    Code:
    user    bondDR    false

    In modifiers.txt:
    Code:
    Path    License to Adventure    (...), Damage Reduction: [10*pref(bondDR)], (...)
 

gausie

D̰͕̝͚̤̥̙̐̇̑͗̒e͍͔͎͈͔ͥ̉̔̅́̈l̠̪̜͓̲ͧ̍̈́͛v̻̾ͤe͗̃ͥ̐̊ͬp̔͒ͪ
Staff member
i may then take this opportunity to add a new column to default.txt for extra information on tags, that way we can encode that the prop is reset on ascension (and also encode props that are reset daily but predate the _ pattern
 
Top