Feedback to: Textpattern CMS 4.6.0 released

colak · 2016-09-13 06:29:24

Bloke wrote #301394:

Fixed, thanks.

It still mentions v4.5. Should that be v4.6?

Bloke · 2016-09-13 06:43:14

colak wrote #301395:

It still mentions v4.5. Should that be v4.6?

OK, really fixed now :-)

JimJoe · 2016-09-14 00:59:49

Thanks for all the work !

johnstephens · 2016-09-14 13:54:58

Thank you! Thank you!

Did something change about the way $variable works? I have the following code working under 4.5.7, but it throws an “undefined index” error on 4.6:

<txp:php>

global $variable;

if ($variable['my_variable']) {
  $variable['my_variable'] = ' '.$variable['my_variable'];
} else {
  $variable['my_variable'] = '';
}

</txp:php>

philwareham wrote #301382:

Hi. That’s planned for a later release, not 4.6.

Is the “ghost” Robert? What happened?

philwareham · 2016-09-14 14:03:06

johnstephens wrote #301456:

Is the “ghost” Robert? What happened?

Ha! That is where we ported the issues across from Google Code, it made all old commenters a ghost for some reason.

Bloke · 2016-09-14 14:04:30

johnstephens wrote #301456:

Did something change about the way $variable works?

Don’t think so, but our warning levels changed so your PHP version may be throwing the warning now, when it didn’t (but should have) under 4.5.7. The error-free way to check if a variable exists in PHP is either:

if (isset($variable['my_variable'])) { ... }

or:

if (!empty($variable['my_variable'])) { ... }

Though be careful of the latter if you expect values that can be evaluated as empty, such as 0, false, null, '', etc.

johnstephens · 2016-09-14 14:23:15

Thank you! In short, “user error”.

I also notice article_custom no longer supports the listform attribute. Easy to work around, though—no problem. I imagine this was a matter of cleanup too?

johnstephens · 2016-09-14 14:27:55

philwareham wrote #301459:

Ha! That is where we ported the issues across from Google Code, it made all old commenters a ghost for some reason.

That’s a relief! I was imagining a cadre of cyber-assassins stalking Team Textpattern and silently rubbing you out, one by one, starting on Github.

Bloke · 2016-09-14 14:28:52

johnstephens wrote #301463:

I also notice article_custom no longer supports the listform attribute… I imagine this was a matter of cleanup too?

Yes, the two tags are handled by the same internal function, and we were a bit lax about which atts went with which. You could legitimately do things like <txp:article id="50" /> and it would let you, but ignore it if you were in article context. Now it’s not permitted and will throw a warning.

Vice-versa, some of the <txp:article /> attributes were allowed in article_custom tags, even though they didn’t make sense. No more.

johnstephens · 2016-09-14 19:20:18

I just noticed that two files are missing from the update directory:

textpattern/update/_to_4.4.0.php
textpattern/update/_to_4.4.1.php

Does this matter?

gaekwad · 2016-09-14 20:03:41

johnstephens wrote #301478:

I just noticed that two files are missing from the update directory:

textpattern/update/_to_4.4.0.php

textpattern/update/_to_4.4.1.php

Does this matter?

Checking these files against the 4.5.7 distribution, there were no actual changes made, so I suspect they were culled as cruft.

Bloke · 2016-09-14 20:10:45

johnstephens wrote #301478:

two files are missing from the update directory… Does this matter?

Nope. They were, as Pete said, cruft. Never used, just had to be there as placeholders because of limitations in the way the upgrade script worked. Since we refactored the upgrade script, they’re no longer required to be there.

Textpattern CMS

Textpattern CMS support forum

#25 2016-09-13 06:29:24

Re: Feedback to: Textpattern CMS 4.6.0 released

Bloke wrote #301394:

#26 2016-09-13 06:43:14

Re: Feedback to: Textpattern CMS 4.6.0 released

colak wrote #301395:

#27 2016-09-14 00:59:49

Re: Feedback to: Textpattern CMS 4.6.0 released

#28 2016-09-14 13:54:58

Re: Feedback to: Textpattern CMS 4.6.0 released

philwareham wrote #301382:

#29 2016-09-14 14:03:06

Re: Feedback to: Textpattern CMS 4.6.0 released

johnstephens wrote #301456:

#30 2016-09-14 14:04:30

Re: Feedback to: Textpattern CMS 4.6.0 released

johnstephens wrote #301456:

#31 2016-09-14 14:23:15

Re: Feedback to: Textpattern CMS 4.6.0 released

#32 2016-09-14 14:27:55

Re: Feedback to: Textpattern CMS 4.6.0 released

philwareham wrote #301459:

#33 2016-09-14 14:28:52

Re: Feedback to: Textpattern CMS 4.6.0 released

johnstephens wrote #301463:

#34 2016-09-14 19:20:18

Re: Feedback to: Textpattern CMS 4.6.0 released

#35 2016-09-14 20:03:41

Re: Feedback to: Textpattern CMS 4.6.0 released

johnstephens wrote #301478:

#36 2016-09-14 20:10:45

Re: Feedback to: Textpattern CMS 4.6.0 released

johnstephens wrote #301478:

Board footer