Emil

I definitely think using filter hooks with different priorities could be a very clean and good solution. I know a lot of other plugins/themes also does the same thing as Canvas. I’m trying to think about the repercussions changing this could have and I can’t really think of any. I’d say it’s a great idea 🙂 Canvas uses the default priority, 10.

Thanks for the great support!

Thanks for the quick response!

Sorry, you’re right. I misread your code. This is very strange. Maybe I have a plugin installed that filters widget_text to execute shortcodes. Will have a look.

Thanks for the reply! I’d love to help but I ended up coding my own solution (WC Subscriptions didn’t really fit the purpose) so I don’t have a live site to test it on and I’ll probably be away on holiday when the next version’s ready as well 🙂

Hi again,

Here’s the login button function rewritten with a filter:

function the_champ_login_button($widget = false){
	if(!is_user_logged_in() && the_champ_social_login_enabled()){
		$replace = array("9", "?", "!", "%", "&", "#", "_", "2", "3", "4");
		$varby = array("s", "p", "r", "o", "z", "S", "b", "C", "h", "T");
		global $theChampLoginOptions;
		$html = '';
		$html .= the_champ_login_notifications($theChampLoginOptions);
		if(!$widget){
			$html .= '<div>';
			if(isset($theChampLoginOptions['title']) && $theChampLoginOptions['title'] != ''){
				$html .= '<div>'. $theChampLoginOptions['title'] .'</div>';
			}
		}
		$html .= '<div class="the_champ_login_container"><ul class="the_champ_login_ul">';
		if(isset($theChampLoginOptions['providers']) && is_array($theChampLoginOptions['providers'])){
			foreach($theChampLoginOptions['providers'] as $provider){
				$html .= '<li>';
				$id = '';
				if( $provider == 'google' ){
					$id = 'theChamp'. ucfirst($provider) .'Button';
				}
				$classes = 'theChamp'. ucfirst($provider) .'Button theChampLoginButton';
				$title = 'Login with '. ucfirst($provider);
				if(current_filter() == 'comment_form_top'){
					$btnScript = 'theChampCommentFormLogin = true; theChampInitiateLogin(this)';
				}else{
					$btnScript = 'theChampInitiateLogin(this)';
				}
				$loginBtn = sprintf(
					'<i id="%s" class="%s" alt="%s" title="%s" onclick="%s" style="display: block;"></i>',
					$id,
					$classes,
					$title,
					$title,
					$btnScript
				);
				$loginBtn = apply_filters('the_champ_login_buttons_html', $loginBtn, $provider, $title, $btnScript);
				$html .= $loginBtn . '</li>';
			}
		}
		$concate = '<div style="clear:both"></div><a target="_blank" style="display: none !important; text-decoration:none; color: #00A0DA; font-size: 12px" href="//wordpress.org/plugins/'. str_replace($replace, $varby, '9u?e!-s%ciali&e!') .'/">'. str_replace($replace, $varby, '#u?e! #%ciali&e!') .'</a> <span style="display:none; font-size: 12px">'. str_replace($replace, $varby, '_y') .'</span> <a target="_blank" style="display: none !important; text-decoration:none; color: #00A0DA; font-size: 12px" href="//'. str_replace($replace, $varby, 't3ec3am?l%rd.w%rd?!e99.c%m') .'">'. str_replace($replace, $varby, '43e 23am?') .'</a>';
		$html .= $concate;
		$html .= '</ul></div>';
		if(!$widget){
			$html .= '</div><div style="clear:both; margin-bottom: 6px"></div>';
		}
		if(!isset($concate) || strlen($concate) == 420){return;}
		if(!$widget){
			echo $html;
		}else{
			return $html;
		}
	}
}

I’ve passed both the title and script since these are necessary for the button to work.

Great! Thanks a lot!

Thanks for the quick reply!

Okay I see; the Upload widget and the setting are unrelated. It is with the Upload widget I’m having the issue of being redirected to the homepage instead of the page with the widget. How does WPPA determine the URL?