refactored transfer ads

This commit is contained in:
a.feteleu 2019-11-16 01:08:31 +03:00
parent 6c82af1940
commit c725da6a9b

View File

@ -14,8 +14,6 @@ class AdvertisementSerializer(serializers.ModelSerializer):
href = serializers.CharField()
site_id = serializers.PrimaryKeyRelatedField(
queryset=Sites.objects.all())
start_at = serializers.DateTimeField(allow_null=True)
expire_at = serializers.DateTimeField(allow_null=True)
class Meta:
"""Meta class."""
@ -24,29 +22,27 @@ class AdvertisementSerializer(serializers.ModelSerializer):
'id',
'href',
'site_id',
'start_at',
'expire_at',
]
def validate(self, data):
data.update({
'old_id': data.pop('id'),
'url': data.pop('href'),
'site': self.get_site(data.pop('site_id')),
'start': data.pop('start_at', None),
'end': data.pop('expire_at', None),
'site_settings': self.get_site_settings(data.pop('site_id')),
})
return data
def create(self, validated_data):
site = validated_data.pop('site')
obj, _ = self.Meta.model.objects.get_or_create(validated_data)
site = validated_data.pop('site_settings')
url = validated_data.get('url')
obj, _ = self.Meta.model.objects.get_or_create(url=url, defaults=validated_data)
if site and site not in obj.sites.all():
obj.sites.add(site)
return obj
def get_site(self, subdomain):
def get_site_settings(self, subdomain):
subdomain = subdomain.country_code_2 if isinstance(subdomain, Sites) else subdomain
qs = SiteSettings.objects.filter(subdomain=subdomain)
if qs.exists():
@ -89,5 +85,5 @@ class AdvertisementImageSerializer(AdvertisementSerializer):
image_url = validated_data.get('image_url')
if advertisement and image_url:
Page.objects.get_or_create(advertisement=advertisement, image_url=image_url, source=Page.MOBILE)
Page.objects.get_or_create(advertisement=advertisement, image_url=image_url, source=Page.WEB)
self.Meta.model.objects.get_or_create(source=Page.MOBILE, **validated_data)
self.Meta.model.objects.get_or_create(source=Page.WEB, **validated_data)