From mboxrd@z Thu Jan 1 00:00:00 1970 From: Larsen To: development@lists.ipfire.org Subject: Re: [PATCH] Mark required input fields with a star Date: Sun, 07 Jun 2015 12:05:36 +0200 Message-ID: In-Reply-To: <1433515431.27049.27.camel@ipfire.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2828465849821085599==" List-Id: --===============2828465849821085599== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit On Fri, 05 Jun 2015 16:43:51 +0200, Michael Tremer wrote: >> + next-server: >> > value='$dhcpsettings{"NEXT_${itf}"}' /> >> - filename: *> /> >> + filename: >> > value='$dhcpsettings{"FILE_${itf}"}' /> >> >> > > There are some hard-coded strings here. These are not newly introduced > but still we should fix them in an other patch later. I am sure you have > come across a couple of them. Didn´t notice before, but made a note on my todo-list for later. On Fri, 05 Jun 2015 16:45:05 +0200, Michael Tremer wrote: > On Thu, 2015-06-04 at 23:46 +0200, Larsen wrote: >> PS: Need help with the TODOs in that patch as I am now web developer >> and where I don´t know how to use an if-clause. > > As these are in the ovpnmain.cgi (as far as I have seen) you should get > in touch with Alex when he is back from holiday. That will be towards > the end of next week. Should not be related to ovpnmain.cgi. I just don´t have the experience how to combine html-code and if-clauses here. Any dev might be able to give a hint here. btw: Will be on vacation myself from june 14th to june 22nd. Lars --===============2828465849821085599==--