Created
November 13, 2009 01:04
-
-
Save juarezpaf/233483 to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| /** | |
| 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; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| /** | |
| 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&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&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