400 bad request for all service messages

Hi we should not return bad request (it happens only when the email does not validate) maybe the dedicated page is not set correctly on main settings? Could you temporary set it to “unstyled page”?

The problem here is the permalink returned:

/%pagename%/?nm=confirmation&nk=36

which translation plugin are you using?

Stefano.

please, check your permalink configuration page an possibly save them again.

Hi Stefano, thank you for taking the time to investigate ! If I set the dedicated page to “unstyled page”, it does work (and I get the admin message included that asks me to style the page).

I use polylang, I took the newsletter shortcode out of the french newsletter page and changed its name so it doesn’t match the main one (I had named them both “newsletter” but the permalink was different as the french one ends with “newsletter-2”).

I tried saving the permalink of the main dedicated page again. I also configured it on the newsletter plugin settings as the dedicated page and saved again. It didn’t work.

Also, I tried unlinking the 2 versions so the main page doesn’t have a translation and it didn’t change anything either.

I don’t really know what else to try, maybe I should do something differently ?

Hi I restore my polylang blog and see if there are things going wrong, maybe with polylang recent releases. But, actually, we just ask WP the permalink of a page so it is something internal.

Stefano.

Hey, I also have this problem and I also use the polylang plugin to translate my website into English.

My page: https://www.ptpodatki.pl/#newsletter

• This reply was modified 4 years, 4 months ago by kiczu.

Hello,

I found out the issue.
The issue is on the The Newsletter Plugin, plugin.php file, method:

/**
     * Returns the Newsletter dedicated page URL or an alternative URL if that page if not
     * configured or not available.
     *
     * @staticvar string $url
     * @return string
     */
    function get_newsletter_page_url($language = '') {

        $page = $this->get_newsletter_page();

        if (!$page || $page->post_status !== 'publish') {
            return $this->build_action_url('m');
        }
        $newsletter_page_url = get_permalink($page->ID);

        if ($language && $newsletter_page_url) {
            if (class_exists('SitePress')) {
                $newsletter_page_url = apply_filters('wpml_permalink', $newsletter_page_url, $language, true);
            }
	        if ( function_exists( 'pll_get_post' ) ) {
                $translated_page = get_permalink( pll_get_post( $page->ID ), $language );
		        if ( $translated_page ) {
			        $newsletter_page_url = $translated_page;
		        }
	        }
        }

        return $newsletter_page_url;
    }

This line (1267)
$translated_page = get_permalink( pll_get_post( $page->ID ), $language );

has the parenthesis on the wrong place. Should be:

$translated_page = get_permalink( pll_get_post( $page->ID, $language ) );

Please fix it!
Btw, thank you for the excellent plugin!

I’m not sure if I did it right, but after correcting the file in this way the whole page stops working and pops up syntax error and unexpected ” ) ” on this line.

Edit: Never mind, that was my bad. The above solution working, very thank you.

• This reply was modified 4 years, 4 months ago by kiczu.

Problem found, patch available in tomorrow version.

Thank you guys!

Thank you!

Thank you everyone ! Back to normal on my part, great ??