search

mrkite / TerraFirma

Mapping for Terraria
http://seancode.com/terrafirma
BSD 2-Clause "Simplified" License
278 stars 61 forks source link

map switch not handled correctly #42

Closed galtgendo closed 8 years ago

galtgendo commented 8 years ago

Again 3.0.9. Switching the maps doesn't seem to reset chest search data - search dialog contains data from the first map the dialog was opened on.

galtgendo commented 8 years ago

...it looks like adding following block at the end of MainWindow::openWorld() in mainwindow.cpp makes things work: 

if (findChests != NULL) {
  delete findChests;
  findChests = NULL;
 }

...though this doesn't strike me as a fully correct solution.

mrkite commented 8 years ago

Oh, good catch. I'll fix that.

mrkite commented 8 years ago

Fixed in newest version.

galtgendo commented 8 years ago

...GitHub notwithstanding, please upload/update source links on your main page too - it seems you've missed those

mrkite commented 8 years ago

I updated them.. but now I've got to update them again :)