anou

Hello,
how/where can I send you the mailjet account email address? I prefer not put here in public forum. Thank you

The use case is that I have a site with 8 languages and I only wanted the popup to show up on one language.

What we could do now is to add the possibility to translate the popup so we can display it on multiple languages in the right language… But that’s another story.

1. line 172: the $check_lg is true by default, so it won’t bother any installation that do not have WPML installed
2. then line 173, I check if WPML exist, and if so, $check_lg becomes false so this way if you haven’t check any language(s) in the Exit Popup Settings page, the popup won’t display anywhere.
3. if you check one or more language(s), the Exit Popup will show up on the given selected language(s)

Finally if you do not do this check, it is like the code added is useless ??

Hope I have been enough clear in my explanation ??

Thank you very much!

Just one mistake: you forgot to use $check_lg in the if() line 183:
if($check_lg && !isset($_COOKIE['viewedExitPopupWP']) &&...

Thank you for your answer, and I already did what you just advised ??

<?php if( !dynamic_sidebar('pub-totalcovering-top') || !dfads( 'groups=712&limit=1' ) ): ?>
    <div class="totalcovering-pub-top"><img src="<?php echo get_stylesheet_directory_uri() . '/images/arch-top.jpg';?>" alt="" /></div>
  <?php endif; ?>

Thanks again

Sorry but it seems that if I remove return_javascript=1' the shortcode outputs nothing.

It is not really a solution but it is maybe a hint to a shared “problem” between dfads shortcode and dynamic_sidebar function ??

Hi @hcabrera

Thx for the info. I’ve made the changes on my local dev without problem with my code.

Only had to comment a part that I fought a lot to find to make my query work :-):

In wpp_i18n_query_where(), I can comment the prepare query part: //$where = $wpdb->prepare( $where, $args );

This is how you learn ?? and thanks to you I’m ready for the next update.

@mailpoet: I think you understand french, so I copy/paste here what my “server master” told me:

SERVER_NAME est égal à “_”, et que c’est invalide pour le dialogue smtp

If you do not understand, I can ask him for a more detailed answer ??

@mailpoet: the 2.7.5 version works for me.
Just add to change in /classes/WJ_Utils.php, line 37-38:
this

if ( isset($_SERVER['SERVER_NAME'] ) && strlen( trim( $_SERVER['SERVER_NAME'] ) ) > 0 ) {
	$domain = strtolower( $_SERVER['SERVER_NAME'] );

by

if ( isset($_SERVER['HTTP_HOST'] ) && strlen( trim( $_SERVER['HTTP_HOST'] ) ) > 0 ) {
	$domain = strtolower( $_SERVER['HTTP_HOST'] );

but that’s kind of another issue ??

Has already said the problem comes from the code line 240 of this file:
/wysija-newsletters/widgets/wysija_nl.php
$args = array_map('sanitize_text_field', $args);

I reversed to the 2.7.2 version of the plugin and i’ll wait for the next release to solve the problem.

Hello,
thanks for your answers, but I finally found that I must disabled the “Ajaxify widget:” option in order to have the right ICL_LANGUAGE_CODE constant.
If enabled, the constant always returns the default language.

So maybe the only difference between us, is that your are testing on localhost and me on a live website…

I will keep digging…

Thanks again.

Until this setting is implemented in the plugin you can always use:
if(is_super_admin){d(...);}
or less restrictive
if(is_user_logged_in()) {d(...);}

Hello,

There is also one “error” for WPML support line 1289, in the __update_views() function:
If no posts was created in the original/default language, the var $id = 0 so the function inserts 0 for postid in the WPP tables.

I think the line should be:
$id = icl_object_id( $id, get_post_type( $id ), true, $sitepress->get_default_language() );
I’ve changed “false” by “true” so the post ID in the language of creation is inserted instead of “0”.

The other “problem” I’ve detected, but it’s maybe only related to my website, is that the code suggested by kuongcomputer above for the __render_popular_post() function doesn’t work for me, because ICL_LANGUAGE_CODE constant is always returning the default language…

After searching in vain, I think it’s maybe a problem of plugins loading order or ajax function call… I’m stuck now. If you have any idea why this constant is not set has it should (it’s ok on front but not in the WPP function), I’m listening!

The consequence of this is that the widget displays posts in the default language even if current language is not the default one.

And thanks for your time and module

Hello mesan00,It was not possible until now to specify the time (Hours and Minutes).
And because your the first one to ask me something, I’ve done it just for you! ??

Hope the others will benefit from this new option in the plugin’s settings.

Just upgrade to 1.5.4, and enjoy!

I think maybe just changing
get_template_directory()
by
get_stylesheet_directory()
should do the trick, because it works for theme and child-theme. And if a child-theme is active it will retrieve the child-theme directory without bothering of the parent theme.