Results 1 to 3 of 3

Thread: 19050: No machete warning against Lianas, lightly tested.

  1. #1
    Feed Reader RSS Bot's Avatar
    Join Date
    Jul 2009
    Posts
    11,565

    RSS 19050: No machete warning against Lianas, lightly tested.

    No machete warning against Lianas, lightly tested.

    by darzil on 2018-12-26 17:07:06

    M /src/net/sourceforge/kolmafia/AreaCombatData.java (view) (diff)
    M /src/net/sourceforge/kolmafia/objectpool/AdventurePool.java (view) (diff)
    M /src/net/sourceforge/kolmafia/objectpool/ItemPool.java (view) (diff)
    M /src/net/sourceforge/kolmafia/request/RelayRequest.java (view) (diff)
    M /src/net/sourceforge/kolmafia/session/ChoiceManager.java (view) (diff)
    M /src/net/sourceforge/kolmafia/session/QuestManager.java (view) (diff)
    Download the latest KolMafia build here.
    Every new revision posted within the hour.
    New EXE builds every Monday.

  2. #2
    Senior Member
    Join Date
    Jan 2014
    Posts
    191

    Default

    I suspect "option=1" is not the correct string to test against, in ChoiceManager.java.

    Actual HTML:
    Code:
    <input type=hidden name=option value=1>
    I was running my ascension script. When it tried to unlock the hidden city areas, this happened:
    It attempted to unlock the areas. hiddenApartmentProgress/hiddenBowlingAlleyProgress/hiddenHospitalProgress/hiddenOfficeProgress were set to 1. None of the areas were actually unlocked.

    What appears to have happened:
    Mafia encountered the unlock NCs. The new ChoiceManager code ran, and set the relevant progress value to 1, incorrectly, because we haven't unlocked the area yet. Some other part of mafia(?) then overrode my "set choiceAdventure783=1" to 6, because it's unlocked? So, it skipped the NC entirely.

  3. #3
    Developer
    Join Date
    Apr 2010
    Posts
    4,998

    Default

    Oops. Try r19051

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •