Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/stats/common.py

Issue 29345242: Noissue - Adapt quotes for compliance with our coding style in sitescripts (Closed)
Patch Set: Created May 29, 2016, 1:26 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sitescripts/stats/bin/pagegenerator.py ('k') | sitescripts/stats/countrycodes.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/stats/common.py
===================================================================
--- a/sitescripts/stats/common.py
+++ b/sitescripts/stats/common.py
@@ -26,9 +26,9 @@
than 150 characters will be still be unique but no longer reversible due to
file system limitations.
"""
- result = re.sub(r"[^\w\.]", lambda match: "-%04x" % ord(match.group(0)), name)
+ result = re.sub(r'[^\w\.]', lambda match: '-%04x' % ord(match.group(0)), name)
if len(result) > 150:
- result = result[:150] + "--%s" % hashlib.md5(result[150:]).hexdigest()
+ result = result[:150] + '--%s' % hashlib.md5(result[150:]).hexdigest()
return result
@@ -36,133 +36,133 @@
"""
This reconstructs a string encoded with filename_encode().
"""
- path = re.sub(r"--[0-9A-Fa-f]{32}", u"\u2026", path)
- path = re.sub(r"-([0-9a-f]{4})", lambda match: unichr(int(match.group(1), 16)), path)
+ path = re.sub(r'--[0-9A-Fa-f]{32}', u'\u2026', path)
+ path = re.sub(r'-([0-9a-f]{4})', lambda match: unichr(int(match.group(1), 16)), path)
return path
basic_fields = [
{
- "name": "day",
- "title": "Days of month",
- "coltitle": "Day",
- "showaverage": True,
- "defaultcount": 31,
- "sort": lambda obj: sorted(obj.items(), key=lambda (k, v): int(k)),
+ 'name': 'day',
+ 'title': 'Days of month',
+ 'coltitle': 'Day',
+ 'showaverage': True,
+ 'defaultcount': 31,
+ 'sort': lambda obj: sorted(obj.items(), key=lambda (k, v): int(k)),
},
{
- "name": "weekday",
- "title": "Days of week",
- "coltitle": "Weekday",
- "showaverage": True,
- "sort": lambda obj: sorted(obj.items(), key=lambda (k, v): int(k)),
- "isspecial": lambda weekday: weekday == 5 or weekday == 6,
+ 'name': 'weekday',
+ 'title': 'Days of week',
+ 'coltitle': 'Weekday',
+ 'showaverage': True,
+ 'sort': lambda obj: sorted(obj.items(), key=lambda (k, v): int(k)),
+ 'isspecial': lambda weekday: weekday == 5 or weekday == 6,
},
{
- "name": "hour",
- "title": "Hours of day",
- "coltitle": "Hour",
- "showaverage": True,
- "sort": lambda obj: sorted(obj.items(), key=lambda (k, v): int(k)),
+ 'name': 'hour',
+ 'title': 'Hours of day',
+ 'coltitle': 'Hour',
+ 'showaverage': True,
+ 'sort': lambda obj: sorted(obj.items(), key=lambda (k, v): int(k)),
},
{
- "name": "country",
- "title": "Countries",
- "coltitle": "Country",
+ 'name': 'country',
+ 'title': 'Countries',
+ 'coltitle': 'Country',
},
{
- "name": "ua",
- "title": "Browsers",
- "coltitle": "Browser",
+ 'name': 'ua',
+ 'title': 'Browsers',
+ 'coltitle': 'Browser',
},
{
- "name": "fullua",
- "title": "Browser versions",
- "coltitle": "Browser version",
+ 'name': 'fullua',
+ 'title': 'Browser versions',
+ 'coltitle': 'Browser version',
},
{
- "name": "referrer",
- "title": "Referrers",
- "coltitle": "Referrer",
+ 'name': 'referrer',
+ 'title': 'Referrers',
+ 'coltitle': 'Referrer',
},
{
- "name": "status",
- "title": "Status codes",
- "coltitle": "Status code",
+ 'name': 'status',
+ 'title': 'Status codes',
+ 'coltitle': 'Status code',
},
{
- "name": "mirror",
- "title": "Download mirrors",
- "coltitle": "Download mirror",
+ 'name': 'mirror',
+ 'title': 'Download mirrors',
+ 'coltitle': 'Download mirror',
},
]
downloader_fields = [
{
- "name": "addonName",
- "title": "Extensions",
- "coltitle": "Extension",
+ 'name': 'addonName',
+ 'title': 'Extensions',
+ 'coltitle': 'Extension',
},
{
- "name": "fullAddon",
- "title": "Extension versions",
- "coltitle": "Extension version",
+ 'name': 'fullAddon',
+ 'title': 'Extension versions',
+ 'coltitle': 'Extension version',
},
{
- "name": "application",
- "title": "Host applications",
- "coltitle": "Host application",
+ 'name': 'application',
+ 'title': 'Host applications',
+ 'coltitle': 'Host application',
},
{
- "name": "fullApplication",
- "title": "Host application versions",
- "coltitle": "Host application version",
+ 'name': 'fullApplication',
+ 'title': 'Host application versions',
+ 'coltitle': 'Host application version',
},
{
- "name": "platform",
- "title": "Platforms",
- "coltitle": "Platform",
+ 'name': 'platform',
+ 'title': 'Platforms',
+ 'coltitle': 'Platform',
},
{
- "name": "fullPlatform",
- "title": "Platform versions",
- "coltitle": "Platform version",
+ 'name': 'fullPlatform',
+ 'title': 'Platform versions',
+ 'coltitle': 'Platform version',
},
{
- "name": "downloadInterval",
- "title": "Download intervals",
- "coltitle": "Download interval",
+ 'name': 'downloadInterval',
+ 'title': 'Download intervals',
+ 'coltitle': 'Download interval',
},
{
- "name": "previousDownload",
- "hidden": True,
+ 'name': 'previousDownload',
+ 'hidden': True,
},
{
- "name": "firstDownload",
- "title": "Initial download",
- "filter": True,
+ 'name': 'firstDownload',
+ 'title': 'Initial download',
+ 'filter': True,
},
{
- "name": "firstInDay",
- "title": "First download this day",
- "filter": True,
+ 'name': 'firstInDay',
+ 'title': 'First download this day',
+ 'filter': True,
},
{
- "name": "firstInWeek",
- "title": "First download this week",
- "filter": True,
+ 'name': 'firstInWeek',
+ 'title': 'First download this week',
+ 'filter': True,
},
{
- "name": "firstInMonth",
- "title": "First download this month",
- "filter": True,
+ 'name': 'firstInMonth',
+ 'title': 'First download this month',
+ 'filter': True,
},
]
install_fields = [
{
- "name": "installType",
- "title": "Install types",
- "coltitle": "Install type",
+ 'name': 'installType',
+ 'title': 'Install types',
+ 'coltitle': 'Install type',
},
]
« no previous file with comments | « sitescripts/stats/bin/pagegenerator.py ('k') | sitescripts/stats/countrycodes.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld