On Thu, 2012-11-15 at 13:17 +0100, Alexander Marx wrote:
>
> I will answer the original here:
>
>
>
>
> Am 13.11.2012 16:08, schrieb Michael Tremer:
> > Why is it necessary to stop the roadwarrior server to manage the IP
> > address pools? I guess a note that all changes require a restart would
> > be sufficient.
> The Button is clickable when server is stopped. That makes sense
> because when you can edit the static networks, they will be written in
> the server.conf and so the ovpn server needs a restart.
> Other Question: WHY AM I NOT ABLE TO EDIT ADVANCED SERVER OPTIONS,
> WHEN SERVER IS RUNNING?!!!
Right, that is exactly the same problem here. I know that you cannot
edit the settings, but the user should still be able to view them.
> > The legend below the table it somewhat messy and unaligned. Don't know
> > if that was already the case before.
> Was the case before, didn't touch that!
Okay. Using the German translation of the WUI does not show this error.
> > The name field accepts values with a comma. Try it out to see the
> > damage.
> > It is also possible to add subnets twice or to add overlapping subnets.
> Fixed.
Well, I cannot have a space in the description, which is not cool.
> > Client configuration page:
> >
> > There is again "net" all over the page. Again "Net name" -> "Name", "IP
> > address pool" -> "Network", "Host address" -> "Assigned IP address".
> > The table is missing a headline.
> Fixed, except Headline. Don#t know if that makes sense here. I can fix
> that, too.
I think so. But it's not a must for me. Let it be as it is right now,
then.
> > "Dynamic OpenVPN server addresspool" -> "Dynamic OpenVPN IP address
> > pool". One could also add the network, just to be sure.
> Fixed.
Could you please add the network to that line?
> > "Net to route" does not really explain what those boxes to. "Behind
> > Client" is not a possible explanation because it depends on the point of
> > view.
> We phoned on that one and agreed to name it "Network behind client"
> And "Network behind IPFIRE".
> If you don't like it now, tell me how to name it!
It's not fine. Indeed it is hard to find a better label for those boxes,
but these are bogus.
> > The select box below is really cool. However, "Green" is spelled mixed
> > case, the other options "BLUE" and "ORANGE" are all upper case. Why is
> > that?
> Fixed. Was in the language file en.pl green was "Green" orange "ORANGE
> and blue BLUE
That's bad.
> > Code review:
> >
> > I see that the entire web user interface is a mess and I see that nobody
> > likes to touch it. But we need to make sure that we don't introduce new
> > problems, because IPFire 2 needs to be maintained for many years and I
> > don't want to make this a hell to ourselves.
> >
> > So. Don't comment in German. Comments in English are a requirement.
> > In case a function replaces an other one, remove the old (and buggy?)
> > function. Make sure that it is clear what arguments a function expects
> > and what the return value is. Use meaningful names for variables. I see
> > that "$a, $b, $c, $d" is much more easy to type, but I find "$byte1,
> > $byte2, ..." better.
> Fixed.
Didn't check that.
> > Also code in a way that is easy to understand. Perl
> > let's users to write many things in one line, but that code does not
> > execute faster. Make clear what you do.
> >
> > Remove the tags which say which lines have been added for the CCD
> > functionality. We use git to track those things.
> Also phoned on that one and agreed to kick the last comments when code
> gets mainstream.
Yeah, that was today :D
> SO: WHICH HALF OF THE MAIL DID I MISS?!
See above.