Skip to content

Instantly share code, notes, and snippets.

@juarezpaf
Created November 13, 2009 01:04
Show Gist options
  • Select an option

  • Save juarezpaf/233483 to your computer and use it in GitHub Desktop.

Select an option

Save juarezpaf/233483 to your computer and use it in GitHub Desktop.
/**
Author: Juarez P. A. Filho @juarezpaf
Inclusão da variável $ad_link na parte de alteração, pois não estava alterando o link corretamente.
Trocar da linha 57 até 96
**/
// Update the ad
if ( $ad_ad != false ) :
$update_ad = $wpdb->query(sprintf("UPDATE $wpdb->imasters_wp_adserver_ads
SET
ad_title = '%s',
ad_ad = '%s',
ad_group_id = %d,
ad_active = %d,
ad_link = '%s',
ad_expiration_date = '%s'
WHERE
ad_id = %d
",
$ad_title,
$ad_ad,
$ad_group_id,
$ad_active,
$ad_link,
$ad_expiration_date,
$ad_id
));
else :
$update_ad = $wpdb->query(sprintf("UPDATE $wpdb->imasters_wp_adserver_ads
SET
ad_title = '%s',
ad_group_id = %d,
ad_active = %d,
ad_link = '%s',
ad_expiration_date = '%s'
WHERE
ad_id = %d
",
$ad_title,
$ad_group_id,
$ad_active,
$ad_link,
$ad_expiration_date,
$ad_id
));
endif;
/**
Author: Juarez P. A. Filho @juarezpaf
Na geração do banner caso o anúncio não tenha link, o plugin estava gerando uma url que não direcionava para nenhum lugar, apenas para uma página em branco, portanto mudei para que caso o usuário não escolha nenhum link o banner fique sem a tag "<a>".
Além dessa alteração, também mudei os parâmetros width e height da imagem para "%s", assim o usuário pode definir tamanhos de banners variáveis, como 150x100% ou 100%x90. Essa foi uma solicitação do cliente que queria poder incluir banners de altura flexível e essa foi a solução mais simples que encontrei.
Trocar da linha 265 até 302
**/
case 'Image' :
if(!empty($ad_link)) :
printf('<a href="%s?a=%d&amp;u=%s"><img src="%s/%s" width="%s" height="%s" alt="" /></a>',
WP_PLUGIN_URL . '/imasters-wp-adserver/imasters-wp-adserver.php',
$ad_id,
$ad_link,
WP_CONTENT_URL . '/ads',
$ad_string,
$ad_dimension_width,
$ad_dimension_height
);
else :
printf('<img src="%s/%s" width="%s" height="%s" alt="" />',
WP_CONTENT_URL . '/ads',
$ad_string,
$ad_dimension_width,
$ad_dimension_height,
$ad_link
);
endif;
break;
case 'Remote' :
if(!empty($ad_link)) :
printf('<a href="%s?a=%d&amp;u=%s"><img src="%s" width="%s" height="%s" alt="" /></a>',
WP_PLUGIN_URL . '/imasters-wp-adserver/imasters-wp-adserver.php',
$ad_id,
$ad_link,
$ad_string,
$ad_dimension_width,
$ad_dimension_height
);
else :
printf('<img src="%s" width="%s" height="%s" alt="" />',
$ad_string,
$ad_dimension_width,
$ad_dimension_height
);
endif;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment