Hi, @Bernhard: Right now I've got a Core 130 branch up and running - I could do this patch for you... Would it be ok if i do? Best, Matthias On 15.04.2019 20:49, Bernhard Bitsch wrote: > Hi Michael, > > I know the "normal way" to send patches. > But I do not have a up-to-date git repo at the moment, reasons exist but aren't of interest for this special case. > I have posted the entire modified dhcpi.cgi in the forum for usage. Wanted just to supply the DevList and Bugzilla with a short and fast solution, hoping not to be to late for the announced emergency update. > > If you want, I can post the entire modified dhcp.cgi to the DevList and someone of the devs can merge it to the git repository. > > - Bernhard > > BTW: I'll setup my local repo again. Further patches for this and other problems and/or developments will obey the guidance from wiki. > >> Gesendet: Montag, 15. April 2019 um 13:12 Uhr >> Von: "Michael Tremer" >> An: "Bernhard Bitsch" >> Cc: "IPFire Development" >> Betreff: Re: Patch for bugzilla #12050, try #2 >> >> Hi Bernhard, >> >> I am happy that you are taking a look at this bug, but this is not how the process works. >> >> We have a guide on how to submit patches to the mailing list: >> >> https://wiki.ipfire.org/devel/submit-patches >> >> There are a thousand reasons why we are doing this as we are doing it. Please follow it. >> >> -Michael >> >> > On 14 Apr 2019, at 00:04, Bernhard Bitsch wrote: >> > >> > Please find appended the solution. >> > >> > Short description of the problem: >> > >> > After adding a new lease, you are in editing mode. >> > All fields are set right, but a second click ('update' ) is necessary. >> > With determining the exact mode 'add/edit' this second click can be avoided for new entries ( new MAC/IP combination). >> > >> > The patch adds determination of add/edit mode. If the MAC/IP pair exists in the list of fixed leases edit mode is selected, add mode is selected else. >> > >> > Sorry, did not produce the patch from git but by diffing changed file and the core 129 original. >> > >> > -Bernhard >> > >> > EDIT: produced an error by blind copy & paste, should correct with this patch. >> >> >