• I was experiencing slowness on my site and started to look at what is happening. Looks like the cause is in RegisterCustomFeedHandler.php file and more specifically in rewriteRuleExists function, which returns always false.

    So something is obviously broken there. The check in that function seems to conflict with Polylang Pro and don’t detect feeds because those have language parameter as well.

    Needless to say, this is a huge performance issue because flushing permalinks is a costly operation.

Viewing 4 replies - 1 through 4 (of 4 total)
  • Thread Starter Timi Wahalahti

    (@sippis)

    I noticed also another thing that causes A LOT of slowness. It’s function wprss_change_publish_button_text that get’s called in gettext filter.

    Using that filter, the plugin generates multiple hundreds of small database queries to get the current post type. Unregistering that hook reduces the database query time 65%.

    Again, needless to say, that this has a huge performance impact. Please find another filter of way to do the same thing.

    Plugin Contributor Miguel Muscat

    (@mekku)

    Hi @sippis

    Thanks for the feedback and suggestions.

    The rewrite rule flushing issue is as you said: a conflict with PolyLang Pro. We’ll need to roll out an update to fix this issue.

    Regarding the gettext filter, we thank you for bringing this to our attention. This is obviously a very inefficient way to change some text, so we’ll definitely need to revise this.

    Thread Starter Timi Wahalahti

    (@sippis)

    Thanks for the prompt reply!

    For now, I’ve fixed the performance issues mentioned just by unhooking those functionalities but that’s not ideal so looking forward to fixes on upcoming releases. Again, thanks – plugin is otherwise very nice!

    Plugin Author Mark Zahra

    (@markzahra)

    @sippis just wanted to chime in and say thanks again for pointing out these issues! We’ll review them in the coming weeks to make sure the plugin performs better.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Permalinks flushed on every page load’ is closed to new replies.