Re: [Vbox4php-develop] Changelog

[Philipp <phi...@gm...>]

Am Sonntag, 11. November 2001 18:10 schrieben Sie:
> On 11 Nov 2001 at 17:26, Philipp Müller wrote:
> > Or in words:
> > I reenabled the possibilty for more than 1 host. To use it, you must
> > update your local config.
>
> hum. I noticed that you changed the templates too, so there's a dropdown
> instead of a textbox. I really liked the way of typing any ip into that
> field to access my remote vbox-server with dynamic ip address using
> vbox4php installed at school. I think that it would be nice to reenable
> this, perhaps using javascript to add hosts to the dropdown dynamically.

Didn't thoght about this possibility.
Hmmm, how about a second line containing a "manual" setting (INPUT 
TYPE="TEXT" as it was before)... is ugly to have 2 lines, I know. 
I don't think that in this case we need to have a Java Script button to first 
at it to the list and then submit it. Just typing it in would be enough-the 
rest could be done in the evaluation of the result (this would spare at least 
one mouse click).

This line should be optional. So how about variables before and after this 
line containing "<!--" and "-->" and displaying them depending on the 
configuration? 
Or with this "variable comments" we could make both possible in the templates.
Would that help? Complex but flexible.

In any case, I will revert it for the blue skin if you like. I would like to 
keep it at least for the default one.

> > I changed the render function, to be able to have more than 1 block. The
> > new one relies on serial blocks (so no nested blocks are allowed) and
> > that each variable in a block has always the same amount of data as the
> > "leading" variable.
>
> No problem. Nesting blocks is what I'll try to implement at next.
> As you did, I think the easiest way of doing block replacements is that for
> every cycle one array element is used. So if there are five values in the
> first variable, there should be five in the second and third (and so on),
> too. (that's the way it was implemented by me)

Yes it is the easiest, but it may lead to some strange results, if the 
programmer is not aware of this (as each variable within a block MUST be of 
this array). 
Maybe there should be at least a check, if the variable is an array type. So 
the replacement will work on arrays only. To make it more tolerant, it may 
check for the count, so it should only replace the elements with the same 
count (but I don't think there is any sense for such a situation at all).

In general, if someone is trying to change the skin, he should know what to 
do and to help him, the documentation should be there.

> Okay, I just removed the whitespaces and linebreaks in
> localconfig.inc.php.bak, because it's really annoying to do that every time
> updating this file.

Sorry for that. I always use my already existing local file, so I didn't 
notice the white spaces.

> I added a short information on the top which may help to get rid of therse
> annoying "headers already sent" messages.

Thanks.

Cheers

Philipp