Path: csiph.com!usenet.pasdenom.info!news.etla.org!news.stack.nl!newsfeed.xs4all.nl!newsfeed4.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
To: python-list@python.org
From: Dennis Lee Bieber <wlfraed@ix.netcom.com>
Subject: Re: Updating a filename's counter value failed each time
Date: Mon, 17 Jun 2013 21:42:00 -0400
Organization: IISS Elusive Unicorn
References: <kpne3k$1066$1@news.ntua.gr> <kpnnfu$t2i$1@reader1.panix.com> <kpno1h$std$1@news.grnet.gr> <MtKvt.47400$ja6.35986@fx18.am4> <kpnsej$8v4$1@reader1.panix.com> <mailman.3496.1371507741.3114.python-list@python.org> <kpo53s$sa6$1@news.grnet.gr>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.3503.1371519731.3114.python-list@python.org>
Lines: 78
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:48586

On Tue, 18 Jun 2013 02:11:55 +0300, ????? <support@superhost.gr> declaimed
the following:

>???? 18/6/2013 1:22 ??, ?/? MRAB ??????:
>> On 17/06/2013 21:44, John Gordon wrote:
>>> In <MtKvt.47400$ja6.35986@fx18.am4> Alister
>>> <alister.ware@ntlworld.com> writes:
>>>
>>>> > #update file's counter if cookie does not exist cur.execute('''UPDATE
>>>> > files SET hits = hits + 1, host = %s, lastvisit =
>>>> > %s WHERE url = %s''', (host, lastvisit, filename) )
>>>> >
>>>> > if cur.rowcount:
>>>> >         print( " database has been affected" )
>>>> >
>>>> > indeed every time i select afilename the message gets printed bu then
>>>> > again noticing the database via phpmyadmin the filename counter is
>>>> > always remaining 0, and not added by +1
>>>
>>>> replase
>>>>  if cur.rowcount:
>>>>          print( " database has been affected" )
>>>
>>>> with print cur.rowcount()
>>>
>>> rowcount isn't a method call; it's just an attribute.  You don't need
>>> the parentheses.
>>>
>> Well, you do need parentheses, it's just that you need them around the
>> 'print':
>>
>> if cur.rowcount:
>>      print(cur.rowcount)
>>
>Okey print( cur.rowcount ) shows each time a select a fialenme from the 
>html table for downlaod that update affected 1 row.
>
>Checking the database records instantly shown that in fact the counter 
>was increased by +1
>
>Runing files.py turn all counter values to 0.
>
>So, update works, its the reload of files.py the deleets the values.
>
># Delete those database records that do not exist as filenames inside 
>the path
>for rec in data:
>	if rec not in filenames:
>		cur.execute('''DELETE FROM files WHERE url = %s''', rec )
>
># Load'em
>for filename in filenames:
>	try:
>		# try to insert the file into the database
>		cur.execute('''INSERT INTO files (url, host, lastvisit) VALUES (%s, 
>%s, %s)''', (filename, host, lastvisit) )
>	except pymysql.ProgrammingError as e:
>		# Insertion failed, file already into database, skip this, go to next 
>filename
>		pass
>
>These lines i dont how but they "manage" not to maintain the counter's value
>
>-- 
>What is now proved was at first only imagined!

	Just out of curiosity... 

	Do you ever COMMIT the changes.

	cur.execute("update anything set something = whatever where that =
this")

without doing a con.commit() is just going to rollback the changes.
-- 
	Wulfraed                 Dennis Lee Bieber         AF6VN
    wlfraed@ix.netcom.com    HTTP://wlfraed.home.netcom.com/