SteveTownsend / SmartHarvestSE

AutoHarvest 2 Skyrim SE mod ported to CommonLibSSE
GNU General Public License v3.0
18 stars 7 forks source link

Stray INI file getting created #488

Closed jtolmer closed 5 months ago

jtolmer commented 5 months ago

It seems that this mod is causing a stray INI file to get created when it auto picks up items without notifications. I reproduced with a minimal mod set running Skyrim 1.6.1170.0:

To reproduce:

The INI file will be named after whatever esp is loaded last. In this minimal repro case, SmartHarvestSE.ini. The contents of the file are

[Interface]
bShowHUDMessages=1

I tried searching to see if this was already reported / a known issue, but didn't find any other reports. My apologies if this is a duplicate report.

SteveTownsend commented 5 months ago

I think this is covered by https://github.com/SteveTownsend/SmartHarvestSE/issues/481

On Sat, Jun 1, 2024 at 6:23 PM Justin T @.***> wrote:

It seems that this mod is causing a stray INI file to get created when it auto picks up items without notifications. I reproduced with a minimal mod set running Skyrim 1.6.1170.0:

  • SKSE
  • Address Library for SKSE
  • SkyUI
  • Smart Harvest

To reproduce:

  • Load a save game
  • Wait for Smart Harvest start-up notifications to finish
  • Shoot an arrow into the ground
  • Wait for arrow to be auto picked up
  • Exit game
  • Notice new, stray INI file in MO2 overwrite directory

The INI file will be named after whatever esp is loaded last. In this minimal repro case, SmartHarvestSE.ini. The contents of the file are

[Interface] bShowHUDMessages=1

I tried searching to see if this was already reported / a known issue, but didn't find any other reports. My apologies if this is a duplicate report.

— Reply to this email directly, view it on GitHub https://github.com/SteveTownsend/SmartHarvestSE/issues/488, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRDNC3WYAD7ZEPHGIB4NGTZFJCWJAVCNFSM6AAAAABIUPX25SVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZDSMZWGY2TMMI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

SteveTownsend commented 5 months ago

Should be fixed once #481 fix is released

jtolmer commented 5 months ago

Thank you for your work on Smart Harvest. It continues to be appreciated.