Aw: Re: Patch for bugzilla #12050, try #2

[Bernhard Bitsch <Bernhard.Bitsch@gmx.de>]

[-- Attachment #1: Type: text/plain, Size: 2113 bytes --]

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" <michael.tremer(a)ipfire.org>
> An: "Bernhard Bitsch" <Bernhard.Bitsch(a)gmx.de>
> Cc: "IPFire Development" <development(a)lists.ipfire.org>
> 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 <Bernhard.Bitsch(a)gmx.de> 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.<dhcp.cgi.patch>
>
>