Textpattern CMS support forum

You are not logged in. Register | Login | Help

#21 2008-01-27 17:38:50

Els
Admin
From: The Netherlands
Registered: 2004-06-06
Posts: 7,458

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

In debugging mode, the pre-flight check now says

clean_url_data_failed: <pre>tag_error <b>&lt;txp:if_section name=&quot;&quot;&gt;</b> -> <b> Notice: Undefined variable: s on line 2772</b></pre>

(Debugging mode outputs a couple of tag errors about mdn_count and ako_ifBrowser, but it doesn’t output the code above…? I can’t find a page where that particular tag error occurs.)

In testing or live mode it is

clean_url_data_failed: <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"

But clean URLs work, and this did not happen in 4.0.5.

Last edited by Els (2008-01-27 17:44:48)

Offline

#22 2008-01-27 17:46:19

ruud
Developer emeritus
From: a galaxy far far away
Registered: 2006-06-04
Posts: 5,068
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

Els, do you by any chance have the rvm_maintenance plugin activated?

Offline

#23 2008-01-27 17:53:53

Els
Admin
From: The Netherlands
Registered: 2004-06-06
Posts: 7,458

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

Ruud

Yes of course. Sorry about that! All is fine when I switch it off.

Offline

#24 2008-01-27 18:10:14

jm
Plugin Author
From: Missoula, MT
Registered: 2005-11-27
Posts: 1,746
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

en-us lang & prefs

  • “Permalink title-like-this (default is TitleLikeThis)?” is incorrect since the default is always title-like-this (an underscore option would be great addition too :)
  • “Allow raw php?” should be “Allow raw PHP?”
  • Could “Automatically append comments to articles?” be set to No by default? Comments are often seen as black magic by beginners – it’d be simpler if the comments were called from the article.

Forms

  • In the default form, <p class="tags"><txp:category1 title="1" link="1" />, <txp:category2 title="1" link="1" /></p> should use a conditional around the categories to avoid a stray comma if cat1 and/or cat2 isn’t used.
  • What is the purpose of <address class="vcard author">&#8212; <span class="fn"><txp:author /></span></address> when there’s no email address?
  • Could the default comments form be converted to a fieldset?
  • Indentation is inconsistent in forms – some forms like comment_form use tabs whereas others like comments_display don’t use any.

Content

  • The default template should not create an about section if it does not create any about content.
  • Instead of the standard lipsum text in “First Post,” it’d be more helpful to have an intro to TXP text with links to TXP resource.

Offline

#25 2008-01-27 18:15:29

RenJonsin
Member
From: Tarpon Springs, FL USA
Registered: 2005-02-06
Posts: 103
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

I updated a local version of the Frontstretch from 4.04 to 4.06. The only warnng message that I got after the update was this:

A problem occured while loading the plugin: ign_password_protect -> User_Warning: Duplicate entry ’1-ign_user_db’ for key 1
insert into maintxp_prefs set prefs_id=1, name=‘ign_user_db’, val=‘txp_users’, html=‘text_input’ on line 81

A problem occured while loading the plugin: ign_password_protect -> User_Warning: Duplicate entry ’1-ign_use_custom’ for key 1
insert into maintxp_prefs set prefs_id=1, name=‘ign_use_custom’, val=‘0’, html=‘yesnoradio’ on line 81

When I click on “show spam” in the comments section, I get this error at the bottom of the page: Warning: opendir(C:\wamp\www\frontstretch/jpcache/cache) [function.opendir]: failed to open dir: Invalid argument in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php(561) : eval()’d code on line 90

[Ruud: all those warnings are plugin related]

All the apostrophes switched to ’

[Ruud: it’s unlikely that this was caused by the upgrade. Can you open a separate topic for this and post your diagnostics. Probably a mismatch between table content and the specified charset in config.php]

I click on the extensions tab and get a page that says “restricted area” That goes away once I turn off rss_admin_db_manager. (As expected. I’ll do an edit later.)

Also, I have lost the tabs for asy_jpcache

Plugins installed:
ajw_admin_workflow: Works as expected
asy_jpcache: No tab on the extensions.
chh_article_custom: Works as expected
chh_article_lib: Works as expected.
hak_tinymce: Clashes with upm_savenew apparently.
ign_password_protect: See issue above. I’m not using that plugin any longer, so I don’t know if it matters.
jcb_columnize: Works as expected
rss_admin_db_manager v4.3: Known issue
rss_admin_show_adv_opts: Works as expected
rss_article_edit: Works as expected
rss_pluginstaller: Works as expected
upm_pending_notify: I was unable to test this as I am using a local version with no email server setup.

upm_savenew v0.4: Notifies that it has posted the new article but doesn’t. errors:
Warning: opendir(C:\wamp\www\frontstretch/jpcache/cache) [function.opendir]: failed to open dir: Invalid argument in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php(561) : eval()’d code on line 90
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 144
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 147
global id:
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 147
13875
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 153

I turned off every plugin but this and it worked fine. I turned on the cache plugin and it worked fine but gave the opendir error.

It seems to be conflicting with the hak_tinymce plugin. I turned on all plugins but that and no errors, turn on hak_tinymce but do not turn on the editor for the article I’m saving new and I get this:

Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 144
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 147
global id:
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 147
13885
Notice: Undefined variable: f in C:\wamp\www\frontstretch\textpattern\lib\txplib_misc.php on line 153

zem_article_info: Works as expected.

[Ruud: thanks very much for all the info on plugins that do/don’t cause problems!]

Offline

#26 2008-01-27 18:25:52

ruud
Developer emeritus
From: a galaxy far far away
Registered: 2006-06-04
Posts: 5,068
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

Bloke wrote:

Since the URLs for the ‘?’ help links are ‘static’ across versions, do you just change them the day 4.0.6 goes live? (e.g. for renaming the “hide e-mail address” & “syndicate excerpt” options?)

The preference names stay the same, it’s just the description that changes. When we change this, it just requires you to update your language strings (preferences -> languages).

Some of the help links (new options?) on the Advanced Prefs go to the “sorry” page. Affected items I noticed are 1) file directory path, 2) plugin cache directory path, 3) widowed titles, 4) mod_deflate compensation, 5) all custom fields.

All added, except custom fields… If someone can write a nice description that I can copy/paste for all custom_field help links, that would be greatly appreciated. Perhaps use some of the info on the <txp:custom_field> textbook page?

Like Els, I’m not a fan of “negative logic” options like “Disallow user images” (‘yes’ by default) which should probably be “Allow user images” (‘no’ by default).

I’ve changed it to “Strip user images”.

Offline

#27 2008-01-27 18:41:49

ruud
Developer emeritus
From: a galaxy far far away
Registered: 2006-06-04
Posts: 5,068
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

jm wrote:

“Permalink title-like-this (default is TitleLikeThis)?” is incorrect since the default is always title-like-this (an underscore option would be great addition too :)

Changed to “Permalink title-like-this (instead of TitleLikeThis)?”

“Allow raw php?” should be “Allow raw PHP?”

Done. Not be me, but done nonetheless.

Could “Automatically append comments to articles?” be set to No by default? Comments are often seen as black magic by beginners – it’d be simpler if the comments were called from the article.

That requires changes to standard forms/pages as well. A non-trivial change. Too late for this release. Technically it’s not a bug or regression.

I’ll wait for feedback from the other devs before commenting on the other things you suggested.

Offline

#28 2008-01-27 18:44:03

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 8,818
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

ruud wrote:

All [help strings] added, except custom fields…

Cool, thanks. Since I raised it, I’ll do custom_fields and send some text to you robbed off the custom_field description on TextBook ;-)

“Remove user images” or “strip images”?

Yeah, either of those will suffice. Much better than “disallow”!


The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.

Txp Builders – finely-crafted code, design and Txp

Offline

#29 2008-01-27 19:12:30

aesop1
Plugin Author
Registered: 2004-09-19
Posts: 142

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

Critical: The admin pages still identify themselves as version 4.05 in the footer. ;)

Unless I accidentally installed 4.05 instead?

Offline

#30 2008-01-27 19:17:47

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 8,818
Website

Re: Feedback to: Help us test the release candidate for the upcoming 4.0.6

aesop1 wrote:

Critical: The admin pages still identify themselves as version 4.05 in the footer. ;)

The revision number hasn’t been increased yet. See 1st post :-)


The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.

Txp Builders – finely-crafted code, design and Txp

Offline

Board footer

Powered by FluxBB