[podcasting] invalid token

Creating and modifying plugins.
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: [podcasting] invalid token

Post by Timbalu »

As far as I know 2k11 has no feed_* tpls, they are in default and last modified two years ago...
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: [podcasting] invalid token

Post by garvinhicking »

Hi!

AFAIR ATOM *requires* xhtml content type; it cannot be switched to HTML(5).

Anyhow, Dirk: nl2p seems to generate this problem; did you disable it for this entry? There's definitely a text markup plugin involved messing with the code, because your original pasted code doesn't contain the buggy </p> nesting.

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Lux
Regular
Posts: 764
Joined: Fri Aug 12, 2005 4:36 pm
Location: Grüt, Zürich, Switzerland
Contact:

Re: [podcasting] invalid token

Post by Lux »

garvinhicking wrote:Anyhow, Dirk: nl2p seems to generate this problem; did you disable it for this entry? There's definitely a text markup plugin involved messing with the code, because your original pasted code doesn't contain the buggy </p> nesting.
No, I did not disable it for the second one. Malte (onli) already stated that it is an error of NL2P, I wait for the fox, ehm, fix.

Cheers

Dirk
onli
Regular
Posts: 3044
Joined: Tue Sep 09, 2008 10:04 pm
Contact:

Re: [podcasting] invalid token

Post by onli »

Lux wrote:Malte (onli) already stated that it is an error of NL2P, I wait for the fox, ehm, fix.
I must admit I'm not sure I'll be able to fix that without a rewrite (for which i'm not sure to have the time). It's a complex piece of code and it feels like i already fixed that bug (probably another, triggering this one, or only trying to fix it). But didn't check the code yet.

My solution for this bug in this particular case (and in the meantime) would be to disable the plugin for this entry.
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: [podcasting] invalid token

Post by Timbalu »

Malte, as this is happening in nested list entries, why not make a regex to avoid the replacement with p in nested cases? This would allow Lux to keep his entries, without disabling nl2p in each single one.
(or merge my 2.13 ISOBR <nl> tag </nl> to also work in p environments, but this would only supply for future or re-edited entries)
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Lux
Regular
Posts: 764
Joined: Fri Aug 12, 2005 4:36 pm
Location: Grüt, Zürich, Switzerland
Contact:

Re: [podcasting] invalid token

Post by Lux »

Timbalu wrote:Malte, as this is happening in nested list entries, why not make a regex to avoid the replacement with p in nested cases? This would allow Lux to keep his entries, without disabling nl2p in each single one.
This is what I thought of.

Cheers

Dirk
onli
Regular
Posts: 3044
Joined: Tue Sep 09, 2008 10:04 pm
Contact:

Re: [podcasting] invalid token

Post by onli »

The whole parser is not regex-based anymore. It goes from right to left, counting newlines, and adding the p-tags where appropriate.
Post Reply