On Jul 10, 7:47 pm, Frantisek Malina <[EMAIL PROTECTED]> wrote:
> Hi all?
> I am trying to update stock in my django shopping cart from the CSV
> feed.
> I am doing this regularly so I need to zero the stock first and loop
> trough the CSV to update.
> The problem is that the following code will only zero the stock and
> will not update.
> How to get around it? Is there a problem with 2 saves?
>
> My CSV
> =====
> SKU, STOCK
> product1,100
> product2,150
> product3,50
> product1,50 //See product1 is here twice, so need to count the stock
> ...
>
> My View
> =====
>
> def update_inventory(request):
> products = Product.objects.order_by('sku')
>
> # Zero all the products stock
> for product in products:
> product.items_in_stock = 0
> product.save()
>
> # Read the feed line by line
> f=open('stocklist.csv', 'r')
> csv=f.readlines()
> f.close()
>
> # Loop over the CSV Product list
> for line in csv:
>
> # Split the SKU/Stock pairs in to a nested list
> csvProductInfo = line.partition(",")
>
> update_sku = csvProductInfo[0]
> update_stock = int(csvProductInfo[2].rstrip("\r
> \n").rstrip("\n"))
>
> # If update_sku is in the db
>
> # Find product by SKU.
> p = Product.objects.get(sku=update_sku)
>
> # Get the stock volume for a particular SKU.
> current_stock = int(p.items_in_stock)
>
> # Update Stock = Stock volume from the DB + stock volume from
> the CSV feed
> p.items_in_stock = current_stock + update_stock
> p.save()
>
> return render_to_response('update_inventory.html', {'products':
> products})
First of all, this is a mailing list. You need to give people time to
respond. You posted your initial request and two updates within two
hours - that's likely to be seen as hassling. People respond if they
know the answer, but two hours just isn't enough time - all the people
who know might be in a different time zone and still asleep, for
example.
Secondly, there isn't anything in particular wrong with the code that
I can see. It could be made more Pythonic - in particular, I'd
recommend using the csv module that comes with Python, rather than
mucking around with readlines and line.partition. Even if you don't
use that, you should at least do this:
f=open('stocklist.csv', 'r')
for line in f:
...etc...
rather than reading the whole thing into a variable and iterating
through it, especially if your CSV file is big.
I'd also recommend putting in some exception handling - what if the
sku value is not found in your database? You should do:
try:
p = Product.objects.get(sku=update_sku)
except Product.DoesNotExist:
// log the error somehow
continue
Finally, you haven't really given us enough information to find the
problem. Have you tried inserting print statements at various points
to see what's going on? You'll see the results in the console, if
you're running the development server. For example, before the final
p.save() I would put:
print p.sku, p.items_in_stock
Try running it again with that line in, and see if that gives you any
useful clues.
--
DR.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django users" group.
To post to this group, send email to django-users@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-users?hl=en
-~----------~----~----~----~------~----~------~--~---
