improvements in product loader
This commit is contained in:
@@ -22,7 +22,7 @@ from companies.models import Company
|
||||
from history.models import HistorySync
|
||||
|
||||
from back_latienda.permissions import IsCreator
|
||||
from .utils import extract_search_filters, find_related_products_v3, find_related_products_v6
|
||||
from .utils import extract_search_filters, find_related_products_v3, find_related_products_v6, product_loader
|
||||
from utils.tag_serializers import TaggitSerializer
|
||||
from utils.tag_filters import ProductTagFilter, ProductOrderFilter
|
||||
|
||||
@@ -64,8 +64,14 @@ def my_products(request):
|
||||
def load_coop_products(request):
|
||||
"""Read CSV file being received
|
||||
Parse it to create products for related Company
|
||||
|
||||
Authenticated user must have a related company
|
||||
"""
|
||||
try:
|
||||
# check company linked to user
|
||||
if request.user.company is None:
|
||||
return Response({"errors":{"details": "Your user has no company to add products to"}})
|
||||
|
||||
csv_file = request.FILES['csv_file']
|
||||
if csv_file.name.endswith('.csv') is not True:
|
||||
logging.error(f"File {csv_file.name} is not a CSV file")
|
||||
@@ -74,67 +80,9 @@ def load_coop_products(request):
|
||||
logging.info(f"Reading contents of {csv_file.name}")
|
||||
decoded_file = csv_file.read().decode('utf-8').splitlines()
|
||||
csv_reader = csv.DictReader(decoded_file, delimiter=',')
|
||||
counter = 0
|
||||
# get company linked to user
|
||||
company_qs = Company.objects.filter(creator=request.user)
|
||||
count = product_loader(csv_reader, request.user)
|
||||
|
||||
if company_qs:
|
||||
company = company_qs.first()
|
||||
else:
|
||||
return Response({"errors":{"details": "Your user has no company to add products to"}})
|
||||
# create historysync instance
|
||||
history = HistorySync.objects.create(company=company, sync_date=datetime.datetime.now(), quantity=len(decoded_file))
|
||||
|
||||
for row in csv_reader:
|
||||
if '' in (row['nombre-producto'], row['descripcion'], row['precio'], row['categoria']):
|
||||
logging.error(f"Required data missing: {row}")
|
||||
continue
|
||||
try:
|
||||
# download image references in csv
|
||||
if row['imagen'].strip() != '':
|
||||
image_url = row['imagen'].strip()
|
||||
response = requests.get(image_url, stream=True)
|
||||
if response.status_code == 200:
|
||||
path = f"{setting.BASE_DIR}media/{row['nombre-producto'].strip()}.{image.url.split('/')[-1]}"
|
||||
logging.info(f"Saving product image to: {path}")
|
||||
new_image = open(path, 'wb')
|
||||
for chunk in response:
|
||||
new_image.write(chunk)
|
||||
new_image.close()
|
||||
else:
|
||||
logging.warning(f"Image URL did not work: {image_url}")
|
||||
new_image = None
|
||||
else:
|
||||
new_image = None
|
||||
|
||||
# TODO: if tags is empty, auto-generate tags
|
||||
|
||||
# assemble instance data
|
||||
product_data = {
|
||||
'id': None if row['id'].strip()=='' else row['id'].strip(),
|
||||
'company': Company.objects.filter(creator=request.user).first(),
|
||||
'name': row['nombre-producto'].strip(),
|
||||
'description': row['descripcion'].strip(),
|
||||
'image': new_image,
|
||||
'url': row['url'].strip(),
|
||||
'price': row['precio'].strip(),
|
||||
'shipping_cost': row['gastos-envio'].strip(),
|
||||
'shipping_terms': row['cond-envio'].strip(),
|
||||
'discount': row['descuento'].strip(),
|
||||
'stock': row['stock'].strip(),
|
||||
'tags': row['tags'].strip(),
|
||||
'category': row['categoria'].strip(),
|
||||
'identifiers': row['identificadores'].strip(),
|
||||
'history': history,
|
||||
'creator': request.user,
|
||||
}
|
||||
|
||||
Product.objects.create(**product_data)
|
||||
logging.info(f"Created Product: {product_data}")
|
||||
counter += 1
|
||||
except Exception as e:
|
||||
logging.error(f"Could not parse {row}")
|
||||
return Response()
|
||||
return Response(f"{count} products registered for {request.user.company_name}")
|
||||
except Exception as e:
|
||||
return Response({"errors": {"details": str(type(e))}}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user