[7.470][BUG][FIXED] Wrong Error When saving without DHCP Server

Hi

If you try to add a static ip mapping for DHCP without selecting any DHCP server you get a wrong message:


The Confd reported an error without providing any details.


maybe already reported ?

thx
Parents
  • Wow, that one was quite subtle.

    Usually, when the WebAdmin starts a Confd transaction in order to change configuration data, it's OK to just do one change after another.  As soon as any one change will fail, the transaction will automatically be terminated, the changes rolled back and an error message prepared for display, which will then automatically be displayed by the handle_errors function.

    But in case, after starting a transaction, the WebAdmin finds out all by himself that something is wrong, without actually causing a Confd error, the handle_errors function does not work - technically, there simply aren't any errors.  In that case, cancelling the transaction, rolling back the changes and displaying an error message must be done manually, and handle_errors must not be called.

    Apparently, this rather rare and exceptional case was missed here, the manual rollback and error display was missing and the automatic handle_errors just told you that no error message could be found...

    Thanks for catching this!
      Ingo
Reply
  • Wow, that one was quite subtle.

    Usually, when the WebAdmin starts a Confd transaction in order to change configuration data, it's OK to just do one change after another.  As soon as any one change will fail, the transaction will automatically be terminated, the changes rolled back and an error message prepared for display, which will then automatically be displayed by the handle_errors function.

    But in case, after starting a transaction, the WebAdmin finds out all by himself that something is wrong, without actually causing a Confd error, the handle_errors function does not work - technically, there simply aren't any errors.  In that case, cancelling the transaction, rolling back the changes and displaying an error message must be done manually, and handle_errors must not be called.

    Apparently, this rather rare and exceptional case was missed here, the manual rollback and error display was missing and the automatic handle_errors just told you that no error message could be found...

    Thanks for catching this!
      Ingo
Children
No Data