Fix for banner link
This commit is contained in:
parent
33fc96ac7c
commit
2ea3193879
|
@ -27,7 +27,7 @@ class AdImage
|
|||
attr_reader :parse_post_date,:parse_unpost_date
|
||||
|
||||
before_validation :add_http
|
||||
validates :out_link, :presence => true, :format => /^(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(([0-9]{1,5})?\/.*)?$/ix
|
||||
validates :out_link, :format => /^(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(([0-9]{1,5})?\/.*)?$/ix, :allow_blank => true
|
||||
|
||||
def parse_post_date=(att)
|
||||
self.post_date = (Date.parse att.gsub(/\s+/, "") rescue nil)
|
||||
|
@ -59,7 +59,7 @@ class AdImage
|
|||
protected
|
||||
|
||||
def add_http
|
||||
unless self.out_link[/^http:\/\//] || self.out_link[/^https:\/\//]
|
||||
unless self.out_link[/^http:\/\//] || self.out_link[/^https:\/\//] || self.out_link.blank?
|
||||
self.out_link = 'http://' + self.out_link
|
||||
end
|
||||
end
|
||||
|
|
|
@ -73,7 +73,8 @@ module ParserCommon
|
|||
res << "time_to_next='#{ad_banner.transition_msec}' "
|
||||
res << "link_open='#{ad_image.link_open}' "
|
||||
# res << "link_url='#{(ad_image.direct_to_after_click?? ad_image.out_link : ad_banner.context) || ' '}' "
|
||||
res << "link_url='#{(ad_image.out_link || ad_banner.context || ' ')}' "
|
||||
res << "link_url='#{(ad_image.out_link)}' "
|
||||
res << "style='cursor:pointer'" if !ad_image.out_link.blank?
|
||||
res << "/>"
|
||||
end
|
||||
res << "</div>"
|
||||
|
|
|
@ -25,6 +25,8 @@ function getTimeout() {
|
|||
|
||||
$(document).ready(function() {
|
||||
$('.slideshow').children('img').click(function()
|
||||
{
|
||||
if($(this).attr('link_url')!='')
|
||||
{
|
||||
if($(this).attr('link_open')=='new_window')
|
||||
{
|
||||
|
@ -34,6 +36,7 @@ $(document).ready(function() {
|
|||
{
|
||||
document.location.href=$(this).attr('link_url')
|
||||
}
|
||||
}
|
||||
})
|
||||
});
|
||||
// Ad Banner FX code [end]
|
||||
|
|
Reference in New Issue