Problems related to today’s update, March 18 2017

That does not sound good. What version of wp-Typography were you running before today’s update? And which PHP version? Are there ans error messages in the debug log?

• This reply was modified 8 years ago by pepe.
• This reply was modified 8 years ago by pepe.

Similar problem with the newest version 4.1.0 here – all very, very slowly after update. Before version: 4.0.2. PHP 7.0 with OPcache and wp-rocket.me as Caching Plugin.

Because WP Rocket has a good caching “engine” – can I disable the wp-Typography built in caching via a config parameter?

Hab mal ein paar Test gemacht: OPCache abschalten – keine positive Ver?nderung, Debug-Log ist leer, Javascript Fehler gibts in der Konsole auch keinen, wenn ich statt PHP 7.0 PHP 5.6 verwende, wirds noch viel, viel langsamer. Wenn ich den wp-rocket.me Cache deaktiviere, wirds auch viiiel langsamer als es eh schon ist.

Es geht mich ja nichts an, aber ich würde das Update zurück ziehen, bis die Ursache gefunden ist, die meisten User spielen mehrere Updates auf einmal ein, manche brauchen l?nger, um raus zu finden, dass es an wp-typography liegt, dass die Seite so langsam geworden ist. K?nnte Stress mit Nutzern ersparen. Habs jetzt aber nicht in mehreren Installationen getestet. Kann schon auch noch ne andere Inkompatibilit?t sein.

Kann ich noch was tun, um zur Probleml?sung beizutragen? Generell find ich das Plugin n?mlich genial – vor allem in Hinblick auf gute Trennungen. Würde es was bringen, wenn man für Trennungen eine bevorzugte Sprache definieren k?nnte oder macht das das Plugin automatisch?

Hat sich was an den Suche/Ersetze Routinen ge?ndert? Da die Trennungen ja prinzipiell erfolgen, muss es irgendwas geben, was sehr lange in der Ausführung braucht …

I’ll be looking into this. In the meantime, I’ve reset the stable version to 4.0.2. If you’ve already updated, you can trigger a downgrade by editing wp-typography.php and setting Version: 4.1.0 to Version: 4.0.1.

Hello pepe,

The problem ist this area

		if ( $this->settings['typo_enable_hyphenation'] ) {
			$transient  = 'typo_php_hyphenator_' . $this->version_hash;
			$hyphenator = $this->_maybe_fix_object( get_transient( $transient ) );

			if ( empty( $hyphenator ) ) {
				$hyphenator = $this->php_typo->get_hyphenator( $this->php_typo->get_settings() );

				// This filter is documented in class-wp-typography.php
				$duration = apply_filters( 'typo_php_typography_caching_duration', 0 );

				// Try again next time.
				$res = set_transient( $transient, $hyphenator, $duration );
			}

			// Let's use it!
			$this->php_typo->set_hyphenator( $hyphenator );
		}

If I comment it out, the speed ist back.

Perhaps the _maybe_fix_object need too much time too

Yes, there are two changes that could be the culprit. Unfortunately, they occur in the same vicinity, so it’s not that easy to see.

@odak71: Could you please try to comment the code in _maybe_fix_object with the exception of the return $object; line and leave the rest in? Then we know if it was one or the other.

Ach und noch was:

Bei einem array:

'ASCII' => array(
			'strlen'     => 'strlen',
			'str_split'  => 'str_split',
			'strtolower' => 'strtolower',
			'substr'     => 'substr',
			'u'          => '', // no regex flag needed.
		),

sollte das letzte Paar danach kein Komma haben:

'ASCII' => array(
			'strlen'     => 'strlen',
			'str_split'  => 'str_split',
			'strtolower' => 'strtolower',
			'substr'     => 'substr',
			'u'          => '' // no regex flag needed.
		),

Dieser Fehler kommt recht h?ufig vor, wei? aber nicht, ob PHP diesen Fehler gn?digerweise ignoriert.

(@odak71: Doch, gem?? Styleguide sollte das letzte Paar ein Komma haben (um leichter weitere Zeilen anfügen zu k?nnen). Die PHP-Syntax erlaubt das, der WordPress-Styleguide erzwingt es.)

• This reply was modified 8 years ago by pepe.

Ah, OK, ist aber auch nicht konsequent durchgezogen, aber das ist ja nebens?chlich.

Es ist diese Funktion:

if ( $this->settings['typo_enable_hyphenation'] ) {
			$transient  = 'typo_php_hyphenator_' . $this->version_hash;
			$hyphenator = $this->_maybe_fix_object( get_transient( $transient ) );

			if ( empty( $hyphenator ) ) {
				$hyphenator = $this->php_typo->get_hyphenator( $this->php_typo->get_settings() );

				// This filter is documented in class-wp-typography.php
				$duration = apply_filters( 'typo_php_typography_caching_duration', 0 );

				// Try again next time.
				$res = set_transient( $transient, $hyphenator, $duration );
			}

			// Let's use it!
			$this->php_typo->set_hyphenator( $hyphenator );
		}

und nicht der _maybe_fix_object – Fix

Die Hash-Funktion k?nnte auch zu zeitaufw?ndig sein, hab jetzt nicht geschaut, wie oft das ganze aufgerufen wird, ich hab nur mal quergeschaut, was es sein k?nnte.

The trailing comma is only for multi-line arrays. That should be consistent.

Anyway, I’ve pushed a beta version to GitHub. @odak71, can you please have a look if https://github.com/mundschenk-at/wp-typography/archive/4.1.x.zip fixes the issue for you?

@dfhemenway, @odak71, it would be great if you could try to install the beta.

Well, I’ve pushed out 4.1.1. Performance should be back to normal (better, in fact).

Apologies for the headache ??

Hello Pepe,

Im back again. Yes, 4.1.1 fixed the problem – thank you very much for the quick solution.