Deleten van config terwijl deze in gebruik is.

wadqc / WAD_Interface

NVKF WAD webinterface

3 stars 1 forks source link

Deleten van config terwijl deze in gebruik is. #49

Closed tcdewit closed 9 years ago

tcdewit commented 9 years ago

Op dit moment is het mogelijk om een module config te deleten terwijl deze nog in gebruik is door een selector. Hiervoor moet in new_analysemodule_cfg.php een extra check komen.

tcdewit commented 9 years ago

Gefixt in commit ae620260eb3ec01207614f6d9ec16f49b86312a9.

wadqc commented 9 years ago

Werkt bij mij niet (dwz config wordt gewoon verwijderd ookal is deze in gebruik). Betreft file WAD_Interface/website/WAD-IQC/database/iqc/new_analysemodule_cfg.php

lijkt foutmelding op regel 203: $result_selector->fetch_object() te geven.
opgelost door op regel 200 het statement $result_selector->close(); te verwijderen.
moet dit statement later dan ergens ingevoegd worden (mijn php kennis schiet te kort)

Enhancement: en moet deze check ook in new_analysemodule.php worden uitgevoerd?

tcdewit commented 9 years ago

Oops.. Dom programmeerfoutje. Gaat elders (bijv new_analyse_module.php) wel goed. Inmiddels gecorrigeerd, thanks!

tcdewit commented 9 years ago

Btw ik neem aan dat ie met het weghalen van close() wel goed werkt?

wadqc commented 9 years ago

klopt

wadqc commented 9 years ago

maar: check werkt bij mij voor verwijderen analyse module (in gebruik in selector) nu niet, de module wordt vrolijk verwijderd.

tcdewit commented 9 years ago

Je hebt mijn laatste versie gebruikt van een uurtje geleden? Vreemd... bij mij werkt ie namelijk wel goed na deze aanpassing; dan wordt het lastig debuggen. Misschien zou je eens wat echo-regels in new_analysemodule_cfg.php kunnen zetten, bijv voor "$field_selector->name" en "sprintf($selector_Stmt,$analysemodule_cfg_ref_key[$i])" ?

wadqc commented 9 years ago

check werkt nu wél voor analyse module config
laatste opmerking hierboven betreft: check werkt nu niet voor analyse module exe Maar ik zie nu dat voor de exe de check ook helemaal niet geimplementeerd is, dus dat klopt. Ik was er voor het gemak vanuitgegaan dat je dezelfde check ook voor de analyse module zelf had geimplementeerd. Misschien een idee?