Go to main content

Textpattern CMS support forum

You are not logged in. Register | Login | Help

#16 2019-12-23 19:07:52

Myusername
Member
Registered: 2019-12-12
Posts: 165

Re: Feedback to: Textpattern CMS 4.8.0-beta released

etc wrote #320675:

Can not reproduce, sorry. The tag has been modified but works as before in my tests. Are you sure to have php enabled?

Yes, it’s all right here except the header tag and the “Please force re-load the page or clear your caches” warning that I insisted on appearing.

I thought maybe I could have missed something by upgrading from 7.7.3 to 7.8.0-beta, so I did an installation from scratch, but still these things keep happening.

Offline

#17 2019-12-23 19:18:58

etc
Developer
Registered: 2010-11-11
Posts: 5,677
Website GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Could you try to reproduce the problem on beta demo site please? We’ve got about 2 hours until the next reset.

Offline

#18 2019-12-23 19:28:38

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 12,455
Website GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Myusername wrote #320676:

Yes, it’s all right here except the header tag

Could you perhaps give us a little clue? Like how you’re trying to use the header: post a relevant bit of code so we can maybe try to replicate what you see. Thank you.

the “Please force re-load the page or clear your caches” warning

Not sure why this is appearing. After you refresh, clear your cache, use a private browsing session or whatever, the message should disappear.

What is the version number in your textpattern/textpattern.js file, your textpattern/index.php (both near the top) and the version string in your txp_prefs table. They should all match.


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

Hire Txp Builders – finely-crafted code, design and Txp

Online

#19 2019-12-24 00:09:55

phiw13
Plugin Author
From: South-Western Japan
Registered: 2004-02-27
Posts: 3,647
Website

Re: Feedback to: Textpattern CMS 4.8.0-beta released

So, those headers.

The txp:header is not working, it seems to me, or at the very least, the <txp:header value="application/xml; charset=utf-8" /> is not.

Test one, local copy of an existing site, on TXP 4.7.3 the xml code displays as expected, but its twin brother (complete copy except for the TXP code) running on the same server with TXP 4.8-dev is displayed as text/html.

Test two: my recently created TXP 4.8-dev playground/test site, pretty much the default install, with just a couple of articles added. As per the multiple forum threads on how to create an xml map with native code, I created a dedicated Section (sitemap) and Page (sitemap-xml.tpl), the dedicated section is set to use the sitemap-xml.tpl page. That page is using – Copy paste! – the Colak code. (with minor adjustments to adapt to local set up)

Accessing txp-dev.local/sitemap gives me this unexpected display (Safari/Firefox – both browsers tell this is a text/html file):

TXP 4.8-dev – MySQL 5.7.x and 8.0.x with Apache/2.4.41 (Unix) PHP/7.3.9

Last edited by phiw13 (2019-12-24 00:11:20)


Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg

Offline

#20 2019-12-24 02:09:10

Myusername
Member
Registered: 2019-12-12
Posts: 165

Re: Feedback to: Textpattern CMS 4.8.0-beta released

phiw13 wrote #320679:

So, those headers.

The txp:header is not working, it seems to me, or at the very least, the <txp:header value="application/xml; charset=utf-8" /> is not.

By my tests an <txp:header> tag is not working at all. Try using two forms: <txp:header value="application/xml;charset=utf-8"/> and <txp:header value="application/json;charset=utf-8" />. unsuccessfully.

Offline

#21 2019-12-24 06:20:33

colak
Admin
From: Cyprus
Registered: 2004-11-20
Posts: 9,386
Website GitHub Mastodon Twitter

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Just a quick one to say that I am also testing the demo site and pasted the following in a new page

<txp:header value="application/xml; charset=utf-8" /><?xml version="1.0" encoding="UTF-8"?>
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
<url><loc><txp:site_url /></loc></url>
<txp:section_list break="">
<url><loc><txp:section url="1" /></loc></url>
</txp:section_list>
<txp:article_custom section="articles"  limit="9999" status>
<url><loc><txp:permlink /></loc><lastmod><txp:modified format="%Y-%m-%dT%H:%M:%SZ" /></lastmod></url>
</txp:article_custom>
</urlset>

Accessing beta-demo.textpattern.co/index.php?s=sitemap returns a 502 error.


Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.

Offline

#22 2019-12-24 06:24:10

phiw13
Plugin Author
From: South-Western Japan
Registered: 2004-02-27
Posts: 3,647
Website

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Did you create site txp-dev.local on your localhost?

Just wanted to say: txp-dev.local is my very local playground.


Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg

Offline

#23 2019-12-24 06:48:08

colak
Admin
From: Cyprus
Registered: 2004-11-20
Posts: 9,386
Website GitHub Mastodon Twitter

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Hi Philippe,
I have done the test on Pete’s demos, both on 4.8beta and the dev ones. Maybe I should try the current stable release. It works just fine on the current stable release.

Last edited by colak (2019-12-24 06:51:02)


Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.

Offline

#24 2019-12-24 06:54:48

phiw13
Plugin Author
From: South-Western Japan
Registered: 2004-02-27
Posts: 3,647
Website

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Somewhere some misunderstanding on the way – on Petes demos site you would looking at https://dev-demo.textpattern.co/index.php?s=sitemap which indeed return s a 502 bad gateway.

( i could use the code you just created, the install was not yet refreshed :-))


Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg

Offline

#25 2019-12-24 07:30:28

phiw13
Plugin Author
From: South-Western Japan
Registered: 2004-02-27
Posts: 3,647
Website

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Something else, for discussion, the theme panel, name column

If a site has only one (1) theme installed, should the “Assign sections” link and the “in use” note be actually displayed? Or should these be suppressed in this case? It looks kinda weird now, and a bit pointless.

And I don’t think a Section can exist without a theme assigned.


Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg

Offline

#26 2019-12-24 09:07:46

etc
Developer
Registered: 2010-11-11
Posts: 5,677
Website GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

Ah, I see. <txp:header value="application/xml; charset=utf-8" /> is missing name attribute, it must be

<txp:header name="Content-Type" value="application/xml; charset=utf-8" />

I will revert name to 4.7-default Content-Type, sorry for the hassle.

Offline

#27 2019-12-24 11:56:38

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 12,455
Website GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

phiw13 wrote #320685:

If a site has only one (1) theme installed, should the “Assign sections” link and the “in use” note be actually displayed?… I don’t think a Section can exist without a theme assigned.

True at the moment. But see this thread on article snippets for a possible future in which sections may be themeless. Then marking them up this way makes sense.

But if we decide not to pursue this way of working then maybe yes we could hide the pills and (perhaps) link. Will give it some thought.


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

Hire Txp Builders – finely-crafted code, design and Txp

Online

#28 2019-12-24 13:29:05

gaekwad
Server grease monkey
From: People's Republic of Cornwall
Registered: 2005-11-19
Posts: 4,731
GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

I’m curious about the 502 errors, I’ll investigate that – thanks for the heads up.

Offline

#29 2019-12-24 13:33:15

gaekwad
Server grease monkey
From: People's Republic of Cornwall
Registered: 2005-11-19
Posts: 4,731
GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

colak wrote #320681:

Accessing beta-demo.textpattern.co/index.php?s=sitemap returns a 502 error.

I can replicate this. Here’s the error in the Nginx log:

2019/12/24 13:31:55 [error] 14090#14090: *9004 upstream sent invalid status "application/xml; charset=utf-8" while reading response header from upstream, client: [IPv4 address], server: beta-demo.textpattern.co, request: "GET /index.php?s=sitemap HTTP/2.0", upstream: "fastcgi://unix:/var/run/php/php7.4-fpm.sock:", host: "beta-demo.textpattern.co"

Investigating…

Offline

#30 2019-12-24 13:36:33

gaekwad
Server grease monkey
From: People's Republic of Cornwall
Registered: 2005-11-19
Posts: 4,731
GitHub

Re: Feedback to: Textpattern CMS 4.8.0-beta released

etc wrote #320689:

Ah, I see. <txp:header value="application/xml; charset=utf-8" /> is missing name attribute, it must be […]

Confirming this fixes the 502 on the beta site.

Offline

Board footer

Powered by FluxBB