Geeklog Forums

Great work, everyone!

Here are my comments:

First of all, a warning for all testers: don't do my mistake. That is, run public_html/docs/install.html before you do anything! I've upgraded my site without using the old config.php and the site stopped working because $_CONF['language files'] was commented out in the databse. Luckily I ran this on a demo site.

Speaking of the old config.php, the upgrade routine ignores its site_name and site_slogan parameters.

Other than that, the new version looks very nice...except one thing: we are once again brought to the English only era. The addition of "text-align:left;" to style.css' body{} hardcoded all Geeklog sites to align to the left, even if they are set to RTL. Please remove this line and trust LTR/RTL to also align the text. Why would anyone ever need a reversed aligning from what LTR/RTL sets itself?

I like the multi-questions polls. But I'm sure the average user would get confused when they click "vote" and only get taken to further questions. The initial button should not be "vote" but "enter poll" because that's what it is really.

Thanks for adding a noreply@ address, but why making it manual instead of using my patch that lets you just use an existing address of the (sub-)admin?

v1.5's function ADMIN_list no longer supports menus (i.e. the $menu_arr parameter) and therefore breaks the site if the site uses any plugin that relies on that parameter.

Here are instructions for any such plugin author (or for testers who already installed the beta while using such plugins):

Change (wherever it is in the plugin's files)

into

("$display .=" is just one example, of course).

Function ADMIN_createMenu is a new v1.5 function. The "" is because this function expects a text line under the menu. You can of course actually deliver such a line, for example:

But in that case be sure to use a language variable and not hardcoded English.

This is great, but can you be more specific? Like in GLUtil's version 1.04 plugin? The plugin installs and uninstalls fine, but the display of the plugin when click on it only gives a somewhat blank page where the information on the center of the page is supposed to be located.

Michael

I've been complaining about ADMIN_list since last August. I have a lib-compat141.php that handles several of the existing differences. I'll be releasing it soon as part of the gllabs plugin. It will make it possible for plugin authors to support GL 1.4.1 and 1.5 with the same code base.

Not everyone will upgrade to 1.5 immediately.

I also plan to create a slightly different list function that isn't so crufty. But I don't know if anyone but me will use it.

That doesn't mean plugin authors shouldn't already use the code I supplied above (manually or by using your file). It's just some lines that will make plugins v1.5 ready (assuming there aren't any other similar changes).


Generally speaking, I can't. This site alone currently lists hundreds upon hundreds of add-ons (almost one hundred of which are officially called plugins). Who knows how many of them (of those that work in v1.4.1) will break v1.5+ sites out there due to this change.

All I can advise you is using a search utility to look for the word "ADMIN_list" (no quotes). For example, WinRAR (which can also open the plugins) contains a search utility.

With that said, I did decide to respect your request and checked GL-Utility 1.0.4 myself. Well, it does not contain the aforementioned word. Therefore, it must be another problem. Dirk, are there any other similar changes (functions that accept less/more parameters in v1.5 than in v1.4.1)?

I've found out that the beta version generates <brxhtml> tags in the user_block, if you enable displaying the full name!

---

Dutch Geeklog sites about camping/hiking:
www.kampeerzaken.nl | www.campersite.nl | www.caravans.nl | www.caravans.net

There were a few of these tags in some non-English language files, e.g. for the Calendar. Those have been fixed in CVS.

Also, the theme has to define an XHTML constant currently or you'll end up with brXHTML tags in all sorts of places. Again, this has already been fixed in CVS.

Could any of these explain what you're seeing?

bye, Dirk

Have a look at the source of my testsite at http://www.boeken-winkel.nl, left column, the second and third block...

Do you mean I'll have to use only XHTML tags (of course!) or I should define the HTML as XHTML in the header?

EDIT: I'll install the CVS version in a few minutes, so here's a fragment of the code:

---

Dutch Geeklog sites about camping/hiking:
www.kampeerzaken.nl | www.campersite.nl | www.caravans.nl | www.caravans.net

Installed the CVS, problem solved

---

Dutch Geeklog sites about camping/hiking:
www.kampeerzaken.nl | www.campersite.nl | www.caravans.nl | www.caravans.net

What I meant was that in 1.5.0b1, a theme had to contain this piece of code in its functions.php file:
The CVS version now defines this automatically if the theme doesn't do it.

bye, Dirk

OK, perfectly clear, Besten Dank dafür

---

Dutch Geeklog sites about camping/hiking:
www.kampeerzaken.nl | www.campersite.nl | www.caravans.nl | www.caravans.net

Seems lika all the ADMIN_list methods have had their menu parameter removed in faviour of the ADMIN_createMenu method.
For example ADMIN_simpleList used in GLUtil 1.0.4

---

Geeklog Plugins: http://plugincms.com