This is an automated notification sent by LCG Savannah.
It relates to:
task #5052, project CDS Invenio
==============================================================================
LATEST MODIFICATIONS of task #5052:
==============================================================================
Update of task #5052 (project cdsware):
Category: None => MiscUtil
Priority: 3 - Low => 1 - Later
Percent Complete: 0% => 50%
_______________________________________________________
Follow-up Comment #2:
Our kwalitee checks do check for these things, albeit a bit less than ideally
than would be possible from within pylint.
Converting these things had better be done manually, and very carefully,
otherwise bad things can happen.
Anyway, we have cleaned many of these in the past, so this task is well
progressed. :)
==============================================================================
OVERVIEW of task #5052:
==============================================================================
URL:
<http://savannah.cern.ch/task/?5052>
Summary: Cleaning run_sql usage
Project: CDS Invenio
Submitted by: skaplun
Submitted on: 2007-06-18 14:31
Should Start On: 2007-06-18 00:00
Should be Finished on: 2007-06-18 00:00
Category: MiscUtil
Priority: 1 - Later
Status: None
Privacy: Public
Percent Complete: 50%
Assigned to: None
Open/Closed: Open
Discussion Lock: Any
Effort: 0.00
_______________________________________________________
Converting every
run_sql("select * from x where user='%s'" % escape_string('foo'))
to
run_sql("select * from x where user=%s", ('foo', ))
Maybe via an automatic tool? ;-)
_______________________________________________________
Follow-up Comments:
-------------------------------------------------------
Date: 2009-03-27 12:03 By: Tibor Simko <simko>
Our kwalitee checks do check for these things, albeit a bit less than ideally
than would be possible from within pylint.
Converting these things had better be done manually, and very carefully,
otherwise bad things can happen.
Anyway, we have cleaned many of these in the past, so this task is well
progressed. :)
-------------------------------------------------------
Date: 2007-06-21 07:57 By: Samuele Kaplun <skaplun>
One idea to enforce this even in the future could be to write a plugin for
pylint for checking CDS feature like this one.
_______________________________________________________
Carbon-Copy List:
CC Address | Comment
------------------------------------+-----------------------------
1576 | -COM-
2195 | -SUB-
==============================================================================
This item URL is:
<http://savannah.cern.ch/task/?5052>
_______________________________________________
Message sent via/by LCG Savannah
http://savannah.cern.ch/
