Boot sequence

  • Resolved olehz

    (@olehz)


    4 years, 4 months ago

    You have recently changed the event for which you load the logic.

    add_action('plugins_loaded', [$ip2location_redirection, 'redirect']);
//add_action('wp', [$ip2location_redirection, 'redirect']);

    Сan you move the event to the settings? Or let it be done with a hook / filter?
    It would be very convenient and add flexibility.
    Or give access to the repo – I’ll do it myself, and you just have to confirm 🙂

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author IP2Location

    (@ip2location)

    4 years, 4 months ago

    Any reason you want to change this?

    Thread Starter olehz

    (@olehz)

    4 years, 4 months ago

    I use two redirectors for a multisite – one by location, the other is internal, the first one should work with yours, the second – the other, before yours worked on the wp event – first, and the other – on the plugins_loaded event. Now both are on plugins_loaded – and problems are possible.
    If you make the event customizable – this will give a huge advantage and the possibility of various combinations.

    aterchin

    (@aterchin)

    4 years, 4 months ago

    I agree with changing it back to ‘wp’ or being able to customize the event. If you ever apply filters to the redirect() method, for example, theme developers can add their own rules without creating a plugin for that method.

    Plugin Author IP2Location

    (@ip2location)

    4 years, 4 months ago

    Let me review again as the current event has a higher priority to load before some cache plugins.

    Thread Starter olehz

    (@olehz)

    4 years, 3 months ago

    Any news on our issue?

    Plugin Author IP2Location

    (@ip2location)

    4 years, 3 months ago

    I will add this back in the next release.

Viewing 6 replies - 1 through 6 (of 6 total)

The topic ‘Boot sequence’ is closed to new replies.