Menu

#1381 Deleting a global variable set creates havoc

Next_Release
fixed
Bug_Report
2023-03-30
2023-02-27
No

If you delete a global variable set the variables keeping the current set/variable are emptied, the choices are also emptied and the dialog looses sync, becoming unmanageable.

Discussion

  • bluehazzard

    bluehazzard - 2023-02-27

    I have many fixes for global variables in my local git Repo. This should be fixed there in addition with some rework of ui. See Forum thread with title New uservarmanager( i can not paste the link here because.i am on mobile an sourceforge prevents pasting things here)
    Can you hold for a moment on this an whait until i updated my remot .git repo On sourceforge?

     
  • Miguel Gimenez

    Miguel Gimenez - 2023-02-27

    OK

     
  • Miguel Gimenez

    Miguel Gimenez - 2023-03-11

    I cannot check in the next few days, will do ASAP.

     
  • Miguel Gimenez

    Miguel Gimenez - 2023-03-16

    It works now.

    The code exporting/importing sets is missing (introduced in [r13218] and [r13224] respectively).

     

    Related

    Commit: [r13218]
    Commit: [r13224]

  • bluehazzard

    bluehazzard - 2023-03-26

    With this patch this functionality should be restored... Can you confirm?
    I will then apply this to trunk...

    [Edit:] Or use the github branch

     

    Last edit: bluehazzard 2023-03-26
  • Miguel Gimenez

    Miguel Gimenez - 2023-03-27

    I have reviewed the last commit in the github branch and looks OK to me,

     
  • bluehazzard

    bluehazzard - 2023-03-28
    • status: open --> fixed
    • assigned_to: bluehazzard
     
  • bluehazzard

    bluehazzard - 2023-03-28

    fixed in [r13245]

     

    Related

    Commit: [r13245]

  • Miguel Gimenez

    Miguel Gimenez - 2023-03-29

    There is a problem now:

    Before the change, if you used $(#wx32.include) with the include field empty then "include" was used instead (the same for the lib field). This is stated in the documentation.

    Now I have to fill explicitly the include field with "include" or use $(#wx32)\include in the project search paths.

    EDIT: the field must be filled with the full path, not just "include"

     

    Last edit: Miguel Gimenez 2023-03-29
  • bluehazzard

    bluehazzard - 2023-03-29

    I really was thinking i tested this before.... Well, never only think but always do... i will fix this at the evening in about 10h) and add some tests...
    Sry for the inconvenience....

     
  • bluehazzard

    bluehazzard - 2023-03-29

    Bug should be fixed, but you have to open the global user variable dialog and save it, without modification. This should fix the config file for the current set.
    This was introduced with the last changes, but only if you open the global user var config dialog. So for normal user this should not trigger after this version

     
  • Miguel Gimenez

    Miguel Gimenez - 2023-03-30

    It works now, thank you.

     

Log in to post a comment.