Fix Configuration.xml - Githubissues

ronie-z-wave commented 7 years ago

Maybe also related to #216 , #228?

I've played a bit around with that issue and figured the following out:

PUT requests 501 from server (not recognized/known) - is OK i think
POST/GET will response 404 if conf/Configuration.xml doesn't exist as I know this file should be added by E-UI right? But untill POST isn't able to add this file it isn't possible in that way ...
If this file exists (added it manually) GET is working well but POST with changed xml won't change the current one. Request is 200 OK with the unchanged Configuration.xml

checked it with postman:

configuration-xml_post_404

configuration-xml_bug

@PoltoS: for me it looks more like a webserver/modhttp problem ...

PoltoS commented 7 years ago

@zwave-mke I propose to replace this Configuration.xml by a special call to ZWaveAPI (tbd in ZWave/index.js) that will save the file in storage as "ExpertUI-Configuration-" (with zway name!!!). Another call will be used to retrive it from the storage.

It will be JSON and not Configuration.xml.

Agree?

vades commented 7 years ago

I agree wit that. Some parts of the UI (configuration, assoc ..) will need to be refactored but JSON is better and faster as XML.

Kosta-Github commented 7 years ago

so, is there a fix available now?

michaelhq commented 6 years ago

same here with 2.3.6, cannot save parameter with fibraro motion sensor It would be nice to have a solution...

vades commented 6 years ago

There is nothing new... :-(

Z-Wave-Me / ExpertUI

Fix Configuration.xml #225