Bug - Not A Bug Default for preference booPeakProgress should be 0, not 100

P

PaladinWhite

Member
I'm not sure I have a full understanding of how property defaults are handled. But if (at least some) properties are set to their defaults on ascension, then the default for booPeakProgress should be 0, instead of its current 100.
 
We could reset that. But, we do reset all the quest flags, and if a particular quest is "unstarted", any ancillary settings are (obviously) not valid yet. But it certainly wouldn't hurt - and would be less confusing to at least one person ;) - if we also reset all the ancillary settings on ascension.
 
Actually, the number is what percentage haunted it is. So a new, unstarted booPeakProgress is actually 100.

booPeakProgress
booPeakProgress tracks the hauntedness level of A-boo Peak. A value of 18, for instance, indicates that A-boo Peak is 18% haunted.
Possible values: 0 - 100
Click to expand...
http://wiki.kolmafia.us/index.php?title=Quest_Tracking_Preferences

Making it start out wrong is not a good thing.
This is not a bug. Making them start wrong is not a [good] Feature Request.
 
Ha ha ha. You are right.

For reference from QuestDatabase.java:

Code: 
	public static void resetQuests()
	{
		for ( int i = 0; i < questLogData.length; ++i )
		{
			// Don't reset Spring Beach Break quests
			if ( !questLogData[ i ][ 0 ].startsWith( "questESl" ) )
			{
				QuestDatabase.setQuestProgress( questLogData[ i ][ 0 ], QuestDatabase.UNSTARTED );
			}
		}
		Preferences.resetToDefault( "manorDrawerCount" );
		Preferences.resetToDefault( "poolSkill" );
		Preferences.resetToDefault( "currentExtremity" );
		Preferences.resetToDefault( "oilPeakProgress" );
		Preferences.resetToDefault( "twinPeakProgress" );
		Preferences.resetToDefault( "booPeakProgress" );
		Preferences.resetToDefault( "desertExploration" );
		Preferences.resetToDefault( "zeppelinProtestors" );
		Preferences.resetToDefault( "middleChamberUnlock" );
		Preferences.resetToDefault( "lowerChamberUnlock" );
		Preferences.resetToDefault( "controlRoomUnlock" );
		Preferences.resetToDefault( "hiddenApartmentProgress" );
		Preferences.resetToDefault( "hiddenHospitalProgress" );
		Preferences.resetToDefault( "hiddenOfficeProgress" );
		Preferences.resetToDefault( "hiddenBowlingAlleyProgress" );
		Preferences.resetToDefault( "blackForestProgress" );
		Preferences.resetToDefault( "nsChallenge1" );
		Preferences.resetToDefault( "nsChallenge2" );
		Preferences.resetToDefault( "nsChallenge3" );
		Preferences.resetToDefault( "nsChallenge4" );
		Preferences.resetToDefault( "nsChallenge5" );
		Preferences.resetToDefault( "nsContestants1" );
		Preferences.resetToDefault( "nsContestants2" );
		Preferences.resetToDefault( "nsContestants3" );
		Preferences.resetToDefault( "nsTowerDoorKeysUsed" );
		Preferences.resetToDefault( "maraisDarkUnlock" );
		Preferences.resetToDefault( "maraisWildlifeUnlock" );
		Preferences.resetToDefault( "maraisCorpseUnlock" );
		Preferences.resetToDefault( "maraisWizardUnlock" );
		Preferences.resetToDefault( "maraisBeaverUnlock" );
		Preferences.resetToDefault( "maraisVillageUnlock" );
		Preferences.resetToDefault( "burnoutsDefeated" );
		Preferences.resetToDefault( "corralUnlocked" );
		Preferences.resetToDefault( "kolhsTotalSchoolSpirited" );
		Preferences.resetToDefault( "haciendaLayout" );
		Preferences.resetToDefault( "spookyravenRecipeUsed" );
		Preferences.resetToDefault( "controlPanel1" );
		Preferences.resetToDefault( "controlPanel2" );
		Preferences.resetToDefault( "controlPanel3" );
		Preferences.resetToDefault( "controlPanel4" );
		Preferences.resetToDefault( "controlPanel5" );
		Preferences.resetToDefault( "controlPanel6" );
		Preferences.resetToDefault( "controlPanel7" );
		Preferences.resetToDefault( "controlPanel8" );
		Preferences.resetToDefault( "controlPanel9" );
		Preferences.resetToDefault( "controlPanelOmega" );
		Preferences.resetToDefault( "SHAWARMAInitiativeUnlocked" );
		Preferences.resetToDefault( "canteenUnlocked" );
		Preferences.resetToDefault( "armoryUnlocked" );
		Preferences.resetToDefault( "writingDesksDefeated" );
		Preferences.resetToDefault( "palindomeDudesDefeated" );
		Preferences.resetToDefault( "warehouseProgress" );
		Preferences.resetToDefault( "dinseyAudienceEngagement" );
		Preferences.resetToDefault( "dinseyFilthLevel" );
		Preferences.resetToDefault( "dinseyFunProgress" );
		Preferences.resetToDefault( "dinseyGatorStenchDamage" );
		Preferences.resetToDefault( "dinseyGarbagePirate" );
		Preferences.resetToDefault( "dinseyNastyBearsDefeated" );
		Preferences.resetToDefault( "dinseyRapidPassEnabled" );
		Preferences.resetToDefault( "dinseyRollercoasterNext" );
		Preferences.resetToDefault( "dinseyRollercoasterStats" );
		Preferences.resetToDefault( "dinseySafetyProtocolsLoose" );
		Preferences.resetToDefault( "dinseySocialJusticeIProgress" );
		Preferences.resetToDefault( "dinseySocialJusticeIIProgress" );
		Preferences.resetToDefault( "dinseyTouristsFed" );
		Preferences.resetToDefault( "dinseyToxicMultiplier" );
	}
As you can see, when we reset quests on Ascension, we do reset all the ancillary settings to their default value - including booPeakProgress.
 
Could chasmBridgeProgress be added to that code?

When you start the quest with the council, chasmBridgeProgress is still 30 until you load the bridge area's page text, which resets it back to zero. This results in an extremely minor and transient display error in my guide which probably nobody noticed. Though, I suppose that is an accurate reading for Actually Ed the Undying.
 
You must log in or register to reply here.
Back
Top