Message ID | 20190924094445.19820-1-stephen@that.guru |
---|---|
State | Accepted |
Headers | show |
Series | [1/2] trivial: Style fixes | expand |
Stephen Finucane <stephen@that.guru> writes: > An assortment of fixes identified through the integration of pre-commit. > > Signed-off-by: Stephen Finucane <stephen@that.guru> > --- > .dockerignore | 1 - > htdocs/css/style.css | 382 +++++++++--------- > lib/packages/jquery/jquery.tablednd.js | 12 +- > manage.py | 5 +- Does it make sense to tweak these files, given that we aren't the original source for them? Regards, Daniel > patchwork/tests/utils.py | 4 +- > .../CVE-2019-13122-e9c63aa346ed15c2.yaml | 2 +- > tox.ini | 2 +- > 7 files changed, 204 insertions(+), 204 deletions(-) > > diff --git a/.dockerignore b/.dockerignore > index 76fdeb7d..216f4ba2 100644 > --- a/.dockerignore > +++ b/.dockerignore > @@ -1,2 +1 @@ > tools/docker/db > - > diff --git a/htdocs/css/style.css b/htdocs/css/style.css > index 9f3ee664..fc816d3d 100644 > --- a/htdocs/css/style.css > +++ b/htdocs/css/style.css > @@ -1,91 +1,91 @@ > h2 { > - font-size: 25px; > - margin: 18px 0 18px 0; > + font-size: 25px; > + margin: 18px 0 18px 0; > } > > h2 a, h2 span { > - font-size: 14px; > - padding-left: 5px; > - font-weight: normal; > + font-size: 14px; > + padding-left: 5px; > + font-weight: normal; > } > > pre { > - line-height: 110%; > - background-color: white; > - border-radius: 0; > + line-height: 110%; > + background-color: white; > + border-radius: 0; > } > > .floaty { > - position: fixed; > - left: 0.1em; > - top: 17em; > + position: fixed; > + left: 0.1em; > + top: 17em; > } > > /* Bootstrap overrides */ > > .navbar-inverse .navbar-brand > a { > - color: white; > + color: white; > } > > .navbar-inverse .navbar-nav > li > a { > - color: #999; > + color: #999; > } > > /* Bootstrap extensions */ > > .navbar-subbrand { > - float: left; > - height: 50px; > - padding: 15px 15px 15px 0; > - font-size: 18px; > - line-height: 20px; > + float: left; > + height: 50px; > + padding: 15px 15px 15px 0; > + font-size: 18px; > + line-height: 20px; > } > > .navbar-subbrand { > - color: #999; > + color: #999; > } > > form { > - padding: 0em; > - margin: 0em; > + padding: 0em; > + margin: 0em; > } > > table { > - border-collapse: collapse; > + border-collapse: collapse; > } > > img { > - border: 0; > + border: 0; > } > > input { > - border: thin solid #909090; > + border: thin solid #909090; > } > > dl dt { > - margin-top: 0.8em; > + margin-top: 0.8em; > } > > #footer { > - padding: 1em; > - font-size: small; > - text-align: center; > - color: #909090; > + padding: 1em; > + font-size: small; > + text-align: center; > + color: #909090; > } > > #footer a { > - color: #909090; > + color: #909090; > } > > /* messages */ > #messages { > - background: #e0e0f0; > - margin: 0.5em 1em 0.0em 0.5em; > - padding: 0.3em; > + background: #e0e0f0; > + margin: 0.5em 1em 0.0em 0.5em; > + padding: 0.3em; > } > > #messages .message { > - color: green; > + color: green; > } > > .filters { > @@ -96,7 +96,7 @@ dl dt { > } > > .filters .checkbox-inline input { > - margin-right: 6px; > + margin-right: 6px; > } > > a.filter-action { > @@ -111,35 +111,35 @@ table.pw-list > thead { > } > > a.colinactive, a.colactive { > - color: black; > - text-decoration: none; > + color: black; > + text-decoration: none; > } > > a.colinactive:hover { > - color: red; > + color: red; > } > > div.filters { > } > > div.patchforms { > - margin-top: 1em; > + margin-top: 1em; > } > > /* list order manipulation */ > > table.patchlist tr.draghover { > - background: #e8e8e8 !important; > + background: #e8e8e8 !important; > } > > .dragging { > - border: thin solid black; > - background: #e8e8e8 !important; > + border: thin solid black; > + background: #e8e8e8 !important; > } > > input#reorder-cancel { > - display: none; > - color: #505050; > + display: none; > + color: #505050; > } > > input#reorder-change { > @@ -147,228 +147,228 @@ input#reorder-change { > > /* list pagination */ > .paginator { > - text-align: right; > - clear: both; > + text-align: right; > + clear: both; > margin: 8px 0 15px; > } > > .paginator .prev-na, > .paginator .next-na { > - font-weight: normal; > - color: #a0a0a0; > + font-weight: normal; > + color: #a0a0a0; > } > > .paginator .prev a, .paginator .prev a:visited, > .paginator .next a, .paginator .next a:visited { > - font-weight: normal; > + font-weight: normal; > } > > .paginator .prev, .paginator .prev-na { margin-right:.5em; } > .paginator .next, .paginator .next-na { margin-left:.5em; } > > .paginator .page a, .paginator .page a:visited, .paginator .curr { > - font-weight: normal; > + font-weight: normal; > padding: 6px 12px; > } > .paginator .curr { > - background-color: #eee; > + background-color: #eee; > border-radius: 4px; > - color: #999; > - font-weight:normal; > + color: #999; > + font-weight:normal; > } > > .thumbnail { > - min-height: 200px; > + min-height: 200px; > } > > /* patch view */ > table.patchmeta th { > - text-align: left; > + text-align: left; > } > > table.patchmeta tr th, table.patchmeta tr td { > - text-align: left; > - padding: 1px 10px; > - vertical-align: middle; > + text-align: left; > + padding: 1px 10px; > + vertical-align: middle; > } > > .patchrelations ul { > - list-style-type: none; > - padding: 0; > - margin: 0; > + list-style-type: none; > + padding: 0; > + margin: 0; > } > > .patchnav { > - padding-left: 1em; > - padding-top: 1em; > + padding-left: 1em; > + padding-top: 1em; > } > > /* checks forms */ > /* TODO(stephenfin): Merge this with 'div.patchform' rules */ > .checks { > - border: 1px solid gray; > - margin: 0.5em 1em; > + border: 1px solid gray; > + margin: 0.5em 1em; > } > > .checks th { > - margin-top: 0em; > - margin-left: -0.6em; > - margin-right: -0.6em; > - padding: 0.3em 0.3em 0.3em 0.6em; > - background-color: #0A0A47; > - color: white; > - font-size: 100%; > - font-weight: normal; > + margin-top: 0em; > + margin-left: -0.6em; > + margin-right: -0.6em; > + padding: 0.3em 0.3em 0.3em 0.6em; > + background-color: #0A0A47; > + color: white; > + font-size: 100%; > + font-weight: normal; > } > > .checks td { > - border-top: 1px solid gray; > - padding: 10px 15px; > - padding-left: 0.2em; > - margin-top: 0em; > + border-top: 1px solid gray; > + padding: 10px 15px; > + padding-left: 0.2em; > + margin-top: 0em; > } > > .checks td a { > - text-decoration: none; > + text-decoration: none; > } > > .checks td a:visited { > - color: #786FB4; > + color: #786FB4; > } > > .checks a:hover { > - text-decoration: underline; > + text-decoration: underline; > } > > .patchlistchecks { > - display: inline-block; > - border-radius: 7px; > - min-width: 0.9em; > - padding: 0 2px; > - text-align: center; > + display: inline-block; > + border-radius: 7px; > + min-width: 0.9em; > + padding: 0 2px; > + text-align: center; > } > > .patchlistchecks.success { > - background-color: #82ca9d; > + background-color: #82ca9d; > } > > .patchlistchecks.warning { > - background-color: #ffc95e; > + background-color: #ffc95e; > } > > .patchlistchecks.fail { > - background-color: #ff5555; > + background-color: #ff5555; > } > > .checks .state { > - font-weight: bold; > - color: #ddd; > + font-weight: bold; > + color: #ddd; > } > > .checks .state.success { > - color: #82ca9d; > + color: #82ca9d; > } > > .checks .state.warning { > - color: #ffe59a; > + color: #ffe59a; > } > > .checks .state.fail { > - color: #f7977a; > + color: #f7977a; > } > > .comment .meta { > - background: #f0f0f0; > - padding: 0.3em 0.5em; > + background: #f0f0f0; > + padding: 0.3em 0.5em; > } > > .comment .content { > - border: 0; > + border: 0; > } > > .patch .content { > - padding: 1em; > + padding: 1em; > } > > .patch-pull-url { > - font-family: "DejaVu Sans Mono", fixed; > + font-family: "DejaVu Sans Mono", fixed; > } > > .quote { > - color: #007f00; > + color: #007f00; > } > > -span.p_header { color: #2e8b57; font-weight: bold; } > -span.p_chunk { color: #a52a2a; font-weight: bold; } > -span.p_context { color: #a020f0; } > -span.p_add { color: #008b8b; } > -span.p_del { color: #6a5acd; } > -span.p_mod { color: #0000ff; } > +span.p_header { color: #2e8b57; font-weight: bold; } > +span.p_chunk { color: #a52a2a; font-weight: bold; } > +span.p_context { color: #a020f0; } > +span.p_add { color: #008b8b; } > +span.p_del { color: #6a5acd; } > +span.p_mod { color: #0000ff; } > > .acked-by { > - color: #2d4566; > + color: #2d4566; > > } > > .nacked-by { > - color: #2d4566; > + color: #2d4566; > > } > > .signed-off-by { > - color: #672d45; > - font-weight: bold; > + color: #672d45; > + font-weight: bold; > } > > .from { > - font-weight: bold; > + font-weight: bold; > } > > /* bundles */ > table.bundlelist { > - margin-top: 2em; > - margin-bottom: 4em; > - margin-left: auto; > - margin-right: auto; > - border: thin solid black; > + margin-top: 2em; > + margin-bottom: 4em; > + margin-left: auto; > + margin-right: auto; > + border: thin solid black; > } > > table.bundlelist th { > - padding-left: 2em; > - padding-right: 2em; > - background: #eeeeee; > - border-bottom: thin solid black; > + padding-left: 2em; > + padding-right: 2em; > + background: #eeeeee; > + border-bottom: thin solid black; > } > > table.bundlelist td > { > - padding-left: 2em; > - padding-right: 2em; > + padding-left: 2em; > + padding-right: 2em; > } > > /* forms that appear for a patch */ > div.patchform { > - border: thin solid #080808; > - padding-left: 0.6em; > - padding-right: 0.6em; > - float: left; > - margin: 0.5em 5em 0.5em 10px; > + border: thin solid #080808; > + padding-left: 0.6em; > + padding-right: 0.6em; > + float: left; > + margin: 0.5em 5em 0.5em 10px; > } > > div.patchform h3 { > - margin-top: 0em; > - margin-left: -0.6em; > - margin-right: -0.6em; > - padding: 0.3em 0.3em 0.3em 0.6em; > - background-color: #222; > - color: #999; > - font-size: 100%; > + margin-top: 0em; > + margin-left: -0.6em; > + margin-right: -0.6em; > + padding: 0.3em 0.3em 0.3em 0.6em; > + background-color: #222; > + color: #999; > + font-size: 100%; > } > > div.patchform ul { > - list-style-type: none; > - padding-left: 0.2em; > - margin-top: 0em; > + list-style-type: none; > + padding-left: 0.2em; > + margin-top: 0em; > } > > /* forms */ > @@ -376,79 +376,79 @@ table.form { > } > > span.help_text { > - font-size: 80%; > + font-size: 80%; > } > > > table.form td { > - padding: 0.6em; > - vertical-align: top; > + padding: 0.6em; > + vertical-align: top; > } > > table.form th.headerrow { > - background: #222; > - color: #999; > - font-weight: bold; > - text-align: center; > + background: #222; > + color: #999; > + font-weight: bold; > + text-align: center; > } > > table.form th { > - text-align: left; > - vertical-align: top; > - padding-top: 0.6em; > + text-align: left; > + vertical-align: top; > + padding-top: 0.6em; > } > > table.form td.form-help { > - font-size: smaller; > - padding-bottom: 1em; > - padding-top: 0em; > + font-size: smaller; > + padding-bottom: 1em; > + padding-top: 0em; > } > > table.form tr td.submitrow { > - border-bottom: 0.2em solid #222; > - color: #333; > - text-align: center; > + border-bottom: 0.2em solid #222; > + color: #333; > + text-align: center; > } > > table.registerform, table.passwordform { > - margin-left: auto; > - margin-right: auto; > + margin-left: auto; > + margin-right: auto; > } > table.loginform { > - margin-left: auto; > - margin-right: auto; > - width: 30em; > + margin-left: auto; > + margin-right: auto; > + width: 30em; > } > > /* form errors */ > .errorlist { > - color: red; > - list-style-type: none; > - padding-left: 0.2em; > - margin: 0em; > + color: red; > + list-style-type: none; > + padding-left: 0.2em; > + margin: 0em; > } > .error { > - color: red; > + color: red; > } > > /* generic table with header columns on the left */ > table.horizontal { > - border-collapse: collapse; > - border: thin solid #e8e8e8; > + border-collapse: collapse; > + border: thin solid #e8e8e8; > } > > table.horizontal th { > - text-align: left; > + text-align: left; > } > > table.horizontal td, table.horizontal th { > - padding: 0.5em 1em; > - border: thin solid #e8e8e8; > + padding: 0.5em 1em; > + border: thin solid #e8e8e8; > } > > /* generic table with header row */ > table.vertical { > - border-collapse: collapse; > + border-collapse: collapse; > } > > /* per suggestions here: https://github.com/twbs/bootstrap/issues/12238 */ > @@ -458,53 +458,53 @@ table.vertical { > .table-extra-condensed > thead > tr > td, > .table-extra-condensed > tbody > tr > td, > .table-extra-condensed > tfoot > tr > td { > - padding: 1px 0.5em 1px 0.5em; > - vertical-align: middle; > + padding: 1px 0.5em 1px 0.5em; > + vertical-align: middle; > } > > table.vertical th { > - background: #222; > - color: #999; > - font-weight: bold; > - text-align: center; > + background: #222; > + color: #999; > + font-weight: bold; > + text-align: center; > } > > table.vertical th, table.vertical td { > - padding: 0.2em 0.8em; > - border: thin solid #e8e8e8; > + padding: 0.2em 0.8em; > + border: thin solid #e8e8e8; > } > > td.numberformat { > - text-align: right; > + text-align: right; > } > > /* boxes */ > div.box { > - border: thin solid #080808; > - margin: 1em; > - padding: 0.5em; > + border: thin solid #080808; > + margin: 1em; > + padding: 0.5em; > } > > div.box h2 { > - background: #222; > - color: #999; > - margin: -0.5em -0.5em 1em; -0.5em; > - padding: 0.3em 0.3em 0.3em 0.6em; > - font-size: 100%; > + background: #222; > + color: #999; > + margin: -0.5em -0.5em 1em; -0.5em; > + padding: 0.3em 0.3em 0.3em 0.6em; > + font-size: 100%; > } > > div.box table.vertical { > - margin-left: auto; > - margin-right: auto; > + margin-left: auto; > + margin-right: auto; > } > > /* columns */ > .leftcol { > - float: left; > - width: 49%; > + float: left; > + width: 49%; > } > > .rightcol { > - float: right; > - width: 49%; > + float: right; > + width: 49%; > } > diff --git a/lib/packages/jquery/jquery.tablednd.js b/lib/packages/jquery/jquery.tablednd.js > index 6cf3a850..f83c6d43 100644 > --- a/lib/packages/jquery/jquery.tablednd.js > +++ b/lib/packages/jquery/jquery.tablednd.js > @@ -5,7 +5,7 @@ > * Licensed like jQuery, see http://docs.jquery.com/License. > * > * Configuration options: > - * > + * > * onDragStyle > * This is the style that is assigned to the row during drag. There are limitations to the styles that can be > * associated with a row (such as you can't assign a border—well you can, but it won't be > @@ -28,14 +28,14 @@ > * Pass a function that will be called when the user starts dragging. The function takes 2 parameters: the > * table and the row which the user has started to drag. > * onAllowDrop > - * Pass a function that will be called as a row is over another row. If the function returns true, allow > + * Pass a function that will be called as a row is over another row. If the function returns true, allow > * dropping on that row, otherwise not. The function takes 2 parameters: the dragged row and the row under > * the cursor. It returns a boolean: true allows the drop, false doesn't allow it. > * scrollAmount > * This is the number of pixels to scroll if the user moves the mouse cursor to the top or bottom of the > * window. The page should automatically scroll up or down as appropriate (tested in IE6, IE7, Safari, FF2, > * FF3 beta) > - * > + * > * Other ways to control behaviour: > * > * Add class="nodrop" to any rows for which you don't want to allow dropping, and class="nodrag" to any rows > @@ -47,7 +47,7 @@ > * > * Known problems: > * - Auto-scoll has some problems with IE7 (it scrolls even when it shouldn't), work-around: set scrollAmount to 0 > - * > + * > * Version 0.2: 2008-02-20 First public version > * Version 0.3: 2008-02-07 Added onDragStart option > * Made the scroll amount configurable (default is 5 as before) > @@ -103,7 +103,7 @@ jQuery.tableDnD = { > var rows = table.rows; //getElementsByTagName("tr") > var config = table.tableDnDConfig; > for (var i=0; i<rows.length; i++) { > - // To make non-draggable rows, add the nodrag class (eg for Category and Header rows) > + // To make non-draggable rows, add the nodrag class (eg for Category and Header rows) > // inspired by John Tarr and Famic > var nodrag = $(rows[i]).hasClass("nodrag"); > if (! nodrag) { //There is no NoDnD attribute on rows I want to drag > @@ -194,7 +194,7 @@ jQuery.tableDnD = { > } > > } > - > + > if (mousePos.y-yOffset < config.scrollAmount) { > window.scrollBy(0, -config.scrollAmount); > } else { > diff --git a/manage.py b/manage.py > index fe6189ee..e4266ade 100755 > --- a/manage.py > +++ b/manage.py > @@ -3,8 +3,9 @@ import os > import sys > > if __name__ == "__main__": > - os.environ.setdefault("DJANGO_SETTINGS_MODULE", > - "patchwork.settings.production") > + os.environ.setdefault( > + "DJANGO_SETTINGS_MODULE", > + "patchwork.settings.production") > > from django.core.management import execute_from_command_line > > diff --git a/patchwork/tests/utils.py b/patchwork/tests/utils.py > index 4ac9afe0..577183d0 100644 > --- a/patchwork/tests/utils.py > +++ b/patchwork/tests/utils.py > @@ -23,8 +23,8 @@ from patchwork.models import SeriesReference > from patchwork.models import State > from patchwork.tests import TEST_PATCH_DIR > > -SAMPLE_DIFF = """--- /dev/null 2011-01-01 00:00:00.000000000 +0800 > -+++ a 2011-01-01 00:00:00.000000000 +0800 > +SAMPLE_DIFF = """--- /dev/null\t2011-01-01 00:00:00.000000000 +0800 > ++++ a\t2011-01-01 00:00:00.000000000 +0800 > @@ -0,0 +1 @@ > +a > """ > diff --git a/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml b/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml > index 48afac05..8b2a9c3d 100644 > --- a/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml > +++ b/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml > @@ -8,4 +8,4 @@ fixes: > many mail systems, including Gmail, and be parsed and stored by > Patchwork. When a user viewed a patch detail page for the patch > with this message id, the script would be run. This is fixed by > - properly escaping the field before it is rendered. > \ No newline at end of file > + properly escaping the field before it is rendered. > diff --git a/tox.ini b/tox.ini > index 0c03857e..617e73cd 100644 > --- a/tox.ini > +++ b/tox.ini > @@ -44,7 +44,7 @@ commands = > [testenv:pep8] > basepython = python2.7 > deps = flake8 > -commands = flake8 {posargs} patchwork > +commands = flake8 {posargs:patchwork manage.py} > > [flake8] > # Some rules are ignored as their use makes the code more difficult to read: > -- > 2.21.0 > > _______________________________________________ > Patchwork mailing list > Patchwork@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/patchwork
On Thu, 2019-09-26 at 23:12 +1000, Daniel Axtens wrote: > Stephen Finucane <stephen@that.guru> writes: > > > An assortment of fixes identified through the integration of pre-commit. > > > > Signed-off-by: Stephen Finucane <stephen@that.guru> > > --- > > .dockerignore | 1 - > > htdocs/css/style.css | 382 +++++++++--------- > > lib/packages/jquery/jquery.tablednd.js | 12 +- > > manage.py | 5 +- > Does it make sense to tweak these files, given that we aren't the > original source for them? I considered that and decided that the trivial nature of the changes (stripping whitespace) was justified. I would avoid more complex linting/refactoring work though, yeah. Stephen > Regards, > Daniel > > > patchwork/tests/utils.py | 4 +- > > .../CVE-2019-13122-e9c63aa346ed15c2.yaml | 2 +- > > tox.ini | 2 +- > > 7 files changed, 204 insertions(+), 204 deletions(-) > > > > diff --git a/.dockerignore b/.dockerignore > > index 76fdeb7d..216f4ba2 100644 > > --- a/.dockerignore > > +++ b/.dockerignore > > @@ -1,2 +1 @@ > > tools/docker/db > > - > > diff --git a/htdocs/css/style.css b/htdocs/css/style.css > > index 9f3ee664..fc816d3d 100644 > > --- a/htdocs/css/style.css > > +++ b/htdocs/css/style.css > > @@ -1,91 +1,91 @@ > > h2 { > > - font-size: 25px; > > - margin: 18px 0 18px 0; > > + font-size: 25px; > > + margin: 18px 0 18px 0; > > } > > > > h2 a, h2 span { > > - font-size: 14px; > > - padding-left: 5px; > > - font-weight: normal; > > + font-size: 14px; > > + padding-left: 5px; > > + font-weight: normal; > > } > > > > pre { > > - line-height: 110%; > > - background-color: white; > > - border-radius: 0; > > + line-height: 110%; > > + background-color: white; > > + border-radius: 0; > > } > > > > .floaty { > > - position: fixed; > > - left: 0.1em; > > - top: 17em; > > + position: fixed; > > + left: 0.1em; > > + top: 17em; > > } > > > > /* Bootstrap overrides */ > > > > .navbar-inverse .navbar-brand > a { > > - color: white; > > + color: white; > > } > > > > .navbar-inverse .navbar-nav > li > a { > > - color: #999; > > + color: #999; > > } > > > > /* Bootstrap extensions */ > > > > .navbar-subbrand { > > - float: left; > > - height: 50px; > > - padding: 15px 15px 15px 0; > > - font-size: 18px; > > - line-height: 20px; > > + float: left; > > + height: 50px; > > + padding: 15px 15px 15px 0; > > + font-size: 18px; > > + line-height: 20px; > > } > > > > .navbar-subbrand { > > - color: #999; > > + color: #999; > > } > > > > form { > > - padding: 0em; > > - margin: 0em; > > + padding: 0em; > > + margin: 0em; > > } > > > > table { > > - border-collapse: collapse; > > + border-collapse: collapse; > > } > > > > img { > > - border: 0; > > + border: 0; > > } > > > > input { > > - border: thin solid #909090; > > + border: thin solid #909090; > > } > > > > dl dt { > > - margin-top: 0.8em; > > + margin-top: 0.8em; > > } > > > > #footer { > > - padding: 1em; > > - font-size: small; > > - text-align: center; > > - color: #909090; > > + padding: 1em; > > + font-size: small; > > + text-align: center; > > + color: #909090; > > } > > > > #footer a { > > - color: #909090; > > + color: #909090; > > } > > > > /* messages */ > > #messages { > > - background: #e0e0f0; > > - margin: 0.5em 1em 0.0em 0.5em; > > - padding: 0.3em; > > + background: #e0e0f0; > > + margin: 0.5em 1em 0.0em 0.5em; > > + padding: 0.3em; > > } > > > > #messages .message { > > - color: green; > > + color: green; > > } > > > > .filters { > > @@ -96,7 +96,7 @@ dl dt { > > } > > > > .filters .checkbox-inline input { > > - margin-right: 6px; > > + margin-right: 6px; > > } > > > > a.filter-action { > > @@ -111,35 +111,35 @@ table.pw-list > thead { > > } > > > > a.colinactive, a.colactive { > > - color: black; > > - text-decoration: none; > > + color: black; > > + text-decoration: none; > > } > > > > a.colinactive:hover { > > - color: red; > > + color: red; > > } > > > > div.filters { > > } > > > > div.patchforms { > > - margin-top: 1em; > > + margin-top: 1em; > > } > > > > /* list order manipulation */ > > > > table.patchlist tr.draghover { > > - background: #e8e8e8 !important; > > + background: #e8e8e8 !important; > > } > > > > .dragging { > > - border: thin solid black; > > - background: #e8e8e8 !important; > > + border: thin solid black; > > + background: #e8e8e8 !important; > > } > > > > input#reorder-cancel { > > - display: none; > > - color: #505050; > > + display: none; > > + color: #505050; > > } > > > > input#reorder-change { > > @@ -147,228 +147,228 @@ input#reorder-change { > > > > /* list pagination */ > > .paginator { > > - text-align: right; > > - clear: both; > > + text-align: right; > > + clear: both; > > margin: 8px 0 15px; > > } > > > > .paginator .prev-na, > > .paginator .next-na { > > - font-weight: normal; > > - color: #a0a0a0; > > + font-weight: normal; > > + color: #a0a0a0; > > } > > > > .paginator .prev a, .paginator .prev a:visited, > > .paginator .next a, .paginator .next a:visited { > > - font-weight: normal; > > + font-weight: normal; > > } > > > > .paginator .prev, .paginator .prev-na { margin-right:.5em; } > > .paginator .next, .paginator .next-na { margin-left:.5em; } > > > > .paginator .page a, .paginator .page a:visited, .paginator .curr { > > - font-weight: normal; > > + font-weight: normal; > > padding: 6px 12px; > > } > > .paginator .curr { > > - background-color: #eee; > > + background-color: #eee; > > border-radius: 4px; > > - color: #999; > > - font-weight:normal; > > + color: #999; > > + font-weight:normal; > > } > > > > .thumbnail { > > - min-height: 200px; > > + min-height: 200px; > > } > > > > /* patch view */ > > table.patchmeta th { > > - text-align: left; > > + text-align: left; > > } > > > > table.patchmeta tr th, table.patchmeta tr td { > > - text-align: left; > > - padding: 1px 10px; > > - vertical-align: middle; > > + text-align: left; > > + padding: 1px 10px; > > + vertical-align: middle; > > } > > > > .patchrelations ul { > > - list-style-type: none; > > - padding: 0; > > - margin: 0; > > + list-style-type: none; > > + padding: 0; > > + margin: 0; > > } > > > > .patchnav { > > - padding-left: 1em; > > - padding-top: 1em; > > + padding-left: 1em; > > + padding-top: 1em; > > } > > > > /* checks forms */ > > /* TODO(stephenfin): Merge this with 'div.patchform' rules */ > > .checks { > > - border: 1px solid gray; > > - margin: 0.5em 1em; > > + border: 1px solid gray; > > + margin: 0.5em 1em; > > } > > > > .checks th { > > - margin-top: 0em; > > - margin-left: -0.6em; > > - margin-right: -0.6em; > > - padding: 0.3em 0.3em 0.3em 0.6em; > > - background-color: #0A0A47; > > - color: white; > > - font-size: 100%; > > - font-weight: normal; > > + margin-top: 0em; > > + margin-left: -0.6em; > > + margin-right: -0.6em; > > + padding: 0.3em 0.3em 0.3em 0.6em; > > + background-color: #0A0A47; > > + color: white; > > + font-size: 100%; > > + font-weight: normal; > > } > > > > .checks td { > > - border-top: 1px solid gray; > > - padding: 10px 15px; > > - padding-left: 0.2em; > > - margin-top: 0em; > > + border-top: 1px solid gray; > > + padding: 10px 15px; > > + padding-left: 0.2em; > > + margin-top: 0em; > > } > > > > .checks td a { > > - text-decoration: none; > > + text-decoration: none; > > } > > > > .checks td a:visited { > > - color: #786FB4; > > + color: #786FB4; > > } > > > > .checks a:hover { > > - text-decoration: underline; > > + text-decoration: underline; > > } > > > > .patchlistchecks { > > - display: inline-block; > > - border-radius: 7px; > > - min-width: 0.9em; > > - padding: 0 2px; > > - text-align: center; > > + display: inline-block; > > + border-radius: 7px; > > + min-width: 0.9em; > > + padding: 0 2px; > > + text-align: center; > > } > > > > .patchlistchecks.success { > > - background-color: #82ca9d; > > + background-color: #82ca9d; > > } > > > > .patchlistchecks.warning { > > - background-color: #ffc95e; > > + background-color: #ffc95e; > > } > > > > .patchlistchecks.fail { > > - background-color: #ff5555; > > + background-color: #ff5555; > > } > > > > .checks .state { > > - font-weight: bold; > > - color: #ddd; > > + font-weight: bold; > > + color: #ddd; > > } > > > > .checks .state.success { > > - color: #82ca9d; > > + color: #82ca9d; > > } > > > > .checks .state.warning { > > - color: #ffe59a; > > + color: #ffe59a; > > } > > > > .checks .state.fail { > > - color: #f7977a; > > + color: #f7977a; > > } > > > > .comment .meta { > > - background: #f0f0f0; > > - padding: 0.3em 0.5em; > > + background: #f0f0f0; > > + padding: 0.3em 0.5em; > > } > > > > .comment .content { > > - border: 0; > > + border: 0; > > } > > > > .patch .content { > > - padding: 1em; > > + padding: 1em; > > } > > > > .patch-pull-url { > > - font-family: "DejaVu Sans Mono", fixed; > > + font-family: "DejaVu Sans Mono", fixed; > > } > > > > .quote { > > - color: #007f00; > > + color: #007f00; > > } > > > > -span.p_header { color: #2e8b57; font-weight: bold; } > > -span.p_chunk { color: #a52a2a; font-weight: bold; } > > -span.p_context { color: #a020f0; } > > -span.p_add { color: #008b8b; } > > -span.p_del { color: #6a5acd; } > > -span.p_mod { color: #0000ff; } > > +span.p_header { color: #2e8b57; font-weight: bold; } > > +span.p_chunk { color: #a52a2a; font-weight: bold; } > > +span.p_context { color: #a020f0; } > > +span.p_add { color: #008b8b; } > > +span.p_del { color: #6a5acd; } > > +span.p_mod { color: #0000ff; } > > > > .acked-by { > > - color: #2d4566; > > + color: #2d4566; > > > > } > > > > .nacked-by { > > - color: #2d4566; > > + color: #2d4566; > > > > } > > > > .signed-off-by { > > - color: #672d45; > > - font-weight: bold; > > + color: #672d45; > > + font-weight: bold; > > } > > > > .from { > > - font-weight: bold; > > + font-weight: bold; > > } > > > > /* bundles */ > > table.bundlelist { > > - margin-top: 2em; > > - margin-bottom: 4em; > > - margin-left: auto; > > - margin-right: auto; > > - border: thin solid black; > > + margin-top: 2em; > > + margin-bottom: 4em; > > + margin-left: auto; > > + margin-right: auto; > > + border: thin solid black; > > } > > > > table.bundlelist th { > > - padding-left: 2em; > > - padding-right: 2em; > > - background: #eeeeee; > > - border-bottom: thin solid black; > > + padding-left: 2em; > > + padding-right: 2em; > > + background: #eeeeee; > > + border-bottom: thin solid black; > > } > > > > table.bundlelist td > > { > > - padding-left: 2em; > > - padding-right: 2em; > > + padding-left: 2em; > > + padding-right: 2em; > > } > > > > /* forms that appear for a patch */ > > div.patchform { > > - border: thin solid #080808; > > - padding-left: 0.6em; > > - padding-right: 0.6em; > > - float: left; > > - margin: 0.5em 5em 0.5em 10px; > > + border: thin solid #080808; > > + padding-left: 0.6em; > > + padding-right: 0.6em; > > + float: left; > > + margin: 0.5em 5em 0.5em 10px; > > } > > > > div.patchform h3 { > > - margin-top: 0em; > > - margin-left: -0.6em; > > - margin-right: -0.6em; > > - padding: 0.3em 0.3em 0.3em 0.6em; > > - background-color: #222; > > - color: #999; > > - font-size: 100%; > > + margin-top: 0em; > > + margin-left: -0.6em; > > + margin-right: -0.6em; > > + padding: 0.3em 0.3em 0.3em 0.6em; > > + background-color: #222; > > + color: #999; > > + font-size: 100%; > > } > > > > div.patchform ul { > > - list-style-type: none; > > - padding-left: 0.2em; > > - margin-top: 0em; > > + list-style-type: none; > > + padding-left: 0.2em; > > + margin-top: 0em; > > } > > > > /* forms */ > > @@ -376,79 +376,79 @@ table.form { > > } > > > > span.help_text { > > - font-size: 80%; > > + font-size: 80%; > > } > > > > > > table.form td { > > - padding: 0.6em; > > - vertical-align: top; > > + padding: 0.6em; > > + vertical-align: top; > > } > > > > table.form th.headerrow { > > - background: #222; > > - color: #999; > > - font-weight: bold; > > - text-align: center; > > + background: #222; > > + color: #999; > > + font-weight: bold; > > + text-align: center; > > } > > > > table.form th { > > - text-align: left; > > - vertical-align: top; > > - padding-top: 0.6em; > > + text-align: left; > > + vertical-align: top; > > + padding-top: 0.6em; > > } > > > > table.form td.form-help { > > - font-size: smaller; > > - padding-bottom: 1em; > > - padding-top: 0em; > > + font-size: smaller; > > + padding-bottom: 1em; > > + padding-top: 0em; > > } > > > > table.form tr td.submitrow { > > - border-bottom: 0.2em solid #222; > > - color: #333; > > - text-align: center; > > + border-bottom: 0.2em solid #222; > > + color: #333; > > + text-align: center; > > } > > > > table.registerform, table.passwordform { > > - margin-left: auto; > > - margin-right: auto; > > + margin-left: auto; > > + margin-right: auto; > > } > > table.loginform { > > - margin-left: auto; > > - margin-right: auto; > > - width: 30em; > > + margin-left: auto; > > + margin-right: auto; > > + width: 30em; > > } > > > > /* form errors */ > > .errorlist { > > - color: red; > > - list-style-type: none; > > - padding-left: 0.2em; > > - margin: 0em; > > + color: red; > > + list-style-type: none; > > + padding-left: 0.2em; > > + margin: 0em; > > } > > .error { > > - color: red; > > + color: red; > > } > > > > /* generic table with header columns on the left */ > > table.horizontal { > > - border-collapse: collapse; > > - border: thin solid #e8e8e8; > > + border-collapse: collapse; > > + border: thin solid #e8e8e8; > > } > > > > table.horizontal th { > > - text-align: left; > > + text-align: left; > > } > > > > table.horizontal td, table.horizontal th { > > - padding: 0.5em 1em; > > - border: thin solid #e8e8e8; > > + padding: 0.5em 1em; > > + border: thin solid #e8e8e8; > > } > > > > /* generic table with header row */ > > table.vertical { > > - border-collapse: collapse; > > + border-collapse: collapse; > > } > > > > /* per suggestions here: https://github.com/twbs/bootstrap/issues/12238 */ > > @@ -458,53 +458,53 @@ table.vertical { > > .table-extra-condensed > thead > tr > td, > > .table-extra-condensed > tbody > tr > td, > > .table-extra-condensed > tfoot > tr > td { > > - padding: 1px 0.5em 1px 0.5em; > > - vertical-align: middle; > > + padding: 1px 0.5em 1px 0.5em; > > + vertical-align: middle; > > } > > > > table.vertical th { > > - background: #222; > > - color: #999; > > - font-weight: bold; > > - text-align: center; > > + background: #222; > > + color: #999; > > + font-weight: bold; > > + text-align: center; > > } > > > > table.vertical th, table.vertical td { > > - padding: 0.2em 0.8em; > > - border: thin solid #e8e8e8; > > + padding: 0.2em 0.8em; > > + border: thin solid #e8e8e8; > > } > > > > td.numberformat { > > - text-align: right; > > + text-align: right; > > } > > > > /* boxes */ > > div.box { > > - border: thin solid #080808; > > - margin: 1em; > > - padding: 0.5em; > > + border: thin solid #080808; > > + margin: 1em; > > + padding: 0.5em; > > } > > > > div.box h2 { > > - background: #222; > > - color: #999; > > - margin: -0.5em -0.5em 1em; -0.5em; > > - padding: 0.3em 0.3em 0.3em 0.6em; > > - font-size: 100%; > > + background: #222; > > + color: #999; > > + margin: -0.5em -0.5em 1em; -0.5em; > > + padding: 0.3em 0.3em 0.3em 0.6em; > > + font-size: 100%; > > } > > > > div.box table.vertical { > > - margin-left: auto; > > - margin-right: auto; > > + margin-left: auto; > > + margin-right: auto; > > } > > > > /* columns */ > > .leftcol { > > - float: left; > > - width: 49%; > > + float: left; > > + width: 49%; > > } > > > > .rightcol { > > - float: right; > > - width: 49%; > > + float: right; > > + width: 49%; > > } > > diff --git a/lib/packages/jquery/jquery.tablednd.js b/lib/packages/jquery/jquery.tablednd.js > > index 6cf3a850..f83c6d43 100644 > > --- a/lib/packages/jquery/jquery.tablednd.js > > +++ b/lib/packages/jquery/jquery.tablednd.js > > @@ -5,7 +5,7 @@ > > * Licensed like jQuery, see http://docs.jquery.com/License. > > * > > * Configuration options: > > - * > > + * > > * onDragStyle > > * This is the style that is assigned to the row during drag. There are limitations to the styles that can be > > * associated with a row (such as you can't assign a border—well you can, but it won't be > > @@ -28,14 +28,14 @@ > > * Pass a function that will be called when the user starts dragging. The function takes 2 parameters: the > > * table and the row which the user has started to drag. > > * onAllowDrop > > - * Pass a function that will be called as a row is over another row. If the function returns true, allow > > + * Pass a function that will be called as a row is over another row. If the function returns true, allow > > * dropping on that row, otherwise not. The function takes 2 parameters: the dragged row and the row under > > * the cursor. It returns a boolean: true allows the drop, false doesn't allow it. > > * scrollAmount > > * This is the number of pixels to scroll if the user moves the mouse cursor to the top or bottom of the > > * window. The page should automatically scroll up or down as appropriate (tested in IE6, IE7, Safari, FF2, > > * FF3 beta) > > - * > > + * > > * Other ways to control behaviour: > > * > > * Add class="nodrop" to any rows for which you don't want to allow dropping, and class="nodrag" to any rows > > @@ -47,7 +47,7 @@ > > * > > * Known problems: > > * - Auto-scoll has some problems with IE7 (it scrolls even when it shouldn't), work-around: set scrollAmount to 0 > > - * > > + * > > * Version 0.2: 2008-02-20 First public version > > * Version 0.3: 2008-02-07 Added onDragStart option > > * Made the scroll amount configurable (default is 5 as before) > > @@ -103,7 +103,7 @@ jQuery.tableDnD = { > > var rows = table.rows; //getElementsByTagName("tr") > > var config = table.tableDnDConfig; > > for (var i=0; i<rows.length; i++) { > > - // To make non-draggable rows, add the nodrag class (eg for Category and Header rows) > > + // To make non-draggable rows, add the nodrag class (eg for Category and Header rows) > > // inspired by John Tarr and Famic > > var nodrag = $(rows[i]).hasClass("nodrag"); > > if (! nodrag) { //There is no NoDnD attribute on rows I want to drag > > @@ -194,7 +194,7 @@ jQuery.tableDnD = { > > } > > > > } > > - > > + > > if (mousePos.y-yOffset < config.scrollAmount) { > > window.scrollBy(0, -config.scrollAmount); > > } else { > > diff --git a/manage.py b/manage.py > > index fe6189ee..e4266ade 100755 > > --- a/manage.py > > +++ b/manage.py > > @@ -3,8 +3,9 @@ import os > > import sys > > > > if __name__ == "__main__": > > - os.environ.setdefault("DJANGO_SETTINGS_MODULE", > > - "patchwork.settings.production") > > + os.environ.setdefault( > > + "DJANGO_SETTINGS_MODULE", > > + "patchwork.settings.production") > > > > from django.core.management import execute_from_command_line > > > > diff --git a/patchwork/tests/utils.py b/patchwork/tests/utils.py > > index 4ac9afe0..577183d0 100644 > > --- a/patchwork/tests/utils.py > > +++ b/patchwork/tests/utils.py > > @@ -23,8 +23,8 @@ from patchwork.models import SeriesReference > > from patchwork.models import State > > from patchwork.tests import TEST_PATCH_DIR > > > > -SAMPLE_DIFF = """--- /dev/null 2011-01-01 00:00:00.000000000 +0800 > > -+++ a 2011-01-01 00:00:00.000000000 +0800 > > +SAMPLE_DIFF = """--- /dev/null\t2011-01-01 00:00:00.000000000 +0800 > > ++++ a\t2011-01-01 00:00:00.000000000 +0800 > > @@ -0,0 +1 @@ > > +a > > """ > > diff --git a/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml b/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml > > index 48afac05..8b2a9c3d 100644 > > --- a/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml > > +++ b/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml > > @@ -8,4 +8,4 @@ fixes: > > many mail systems, including Gmail, and be parsed and stored by > > Patchwork. When a user viewed a patch detail page for the patch > > with this message id, the script would be run. This is fixed by > > - properly escaping the field before it is rendered. > > \ No newline at end of file > > + properly escaping the field before it is rendered. > > diff --git a/tox.ini b/tox.ini > > index 0c03857e..617e73cd 100644 > > --- a/tox.ini > > +++ b/tox.ini > > @@ -44,7 +44,7 @@ commands = > > [testenv:pep8] > > basepython = python2.7 > > deps = flake8 > > -commands = flake8 {posargs} patchwork > > +commands = flake8 {posargs:patchwork manage.py} > > > > [flake8] > > # Some rules are ignored as their use makes the code more difficult to read: > > -- > > 2.21.0 > > > > _______________________________________________ > > Patchwork mailing list > > Patchwork@lists.ozlabs.org > > https://lists.ozlabs.org/listinfo/patchwork
On Fri, 2019-09-27 at 08:55 +0100, Stephen Finucane wrote: > On Thu, 2019-09-26 at 23:12 +1000, Daniel Axtens wrote: > > Stephen Finucane <stephen@that.guru> writes: > > > > > An assortment of fixes identified through the integration of pre-commit. > > > > > > Signed-off-by: Stephen Finucane <stephen@that.guru> > > > --- > > > .dockerignore | 1 - > > > htdocs/css/style.css | 382 +++++++++--------- > > > lib/packages/jquery/jquery.tablednd.js | 12 +- > > > manage.py | 5 +- > > Does it make sense to tweak these files, given that we aren't the > > original source for them? > > I considered that and decided that the trivial nature of the changes > (stripping whitespace) was justified. I would avoid more complex > linting/refactoring work though, yeah. > > Stephen I've gone ahead and applied this. Stephen
diff --git a/.dockerignore b/.dockerignore index 76fdeb7d..216f4ba2 100644 --- a/.dockerignore +++ b/.dockerignore @@ -1,2 +1 @@ tools/docker/db - diff --git a/htdocs/css/style.css b/htdocs/css/style.css index 9f3ee664..fc816d3d 100644 --- a/htdocs/css/style.css +++ b/htdocs/css/style.css @@ -1,91 +1,91 @@ h2 { - font-size: 25px; - margin: 18px 0 18px 0; + font-size: 25px; + margin: 18px 0 18px 0; } h2 a, h2 span { - font-size: 14px; - padding-left: 5px; - font-weight: normal; + font-size: 14px; + padding-left: 5px; + font-weight: normal; } pre { - line-height: 110%; - background-color: white; - border-radius: 0; + line-height: 110%; + background-color: white; + border-radius: 0; } .floaty { - position: fixed; - left: 0.1em; - top: 17em; + position: fixed; + left: 0.1em; + top: 17em; } /* Bootstrap overrides */ .navbar-inverse .navbar-brand > a { - color: white; + color: white; } .navbar-inverse .navbar-nav > li > a { - color: #999; + color: #999; } /* Bootstrap extensions */ .navbar-subbrand { - float: left; - height: 50px; - padding: 15px 15px 15px 0; - font-size: 18px; - line-height: 20px; + float: left; + height: 50px; + padding: 15px 15px 15px 0; + font-size: 18px; + line-height: 20px; } .navbar-subbrand { - color: #999; + color: #999; } form { - padding: 0em; - margin: 0em; + padding: 0em; + margin: 0em; } table { - border-collapse: collapse; + border-collapse: collapse; } img { - border: 0; + border: 0; } input { - border: thin solid #909090; + border: thin solid #909090; } dl dt { - margin-top: 0.8em; + margin-top: 0.8em; } #footer { - padding: 1em; - font-size: small; - text-align: center; - color: #909090; + padding: 1em; + font-size: small; + text-align: center; + color: #909090; } #footer a { - color: #909090; + color: #909090; } /* messages */ #messages { - background: #e0e0f0; - margin: 0.5em 1em 0.0em 0.5em; - padding: 0.3em; + background: #e0e0f0; + margin: 0.5em 1em 0.0em 0.5em; + padding: 0.3em; } #messages .message { - color: green; + color: green; } .filters { @@ -96,7 +96,7 @@ dl dt { } .filters .checkbox-inline input { - margin-right: 6px; + margin-right: 6px; } a.filter-action { @@ -111,35 +111,35 @@ table.pw-list > thead { } a.colinactive, a.colactive { - color: black; - text-decoration: none; + color: black; + text-decoration: none; } a.colinactive:hover { - color: red; + color: red; } div.filters { } div.patchforms { - margin-top: 1em; + margin-top: 1em; } /* list order manipulation */ table.patchlist tr.draghover { - background: #e8e8e8 !important; + background: #e8e8e8 !important; } .dragging { - border: thin solid black; - background: #e8e8e8 !important; + border: thin solid black; + background: #e8e8e8 !important; } input#reorder-cancel { - display: none; - color: #505050; + display: none; + color: #505050; } input#reorder-change { @@ -147,228 +147,228 @@ input#reorder-change { /* list pagination */ .paginator { - text-align: right; - clear: both; + text-align: right; + clear: both; margin: 8px 0 15px; } .paginator .prev-na, .paginator .next-na { - font-weight: normal; - color: #a0a0a0; + font-weight: normal; + color: #a0a0a0; } .paginator .prev a, .paginator .prev a:visited, .paginator .next a, .paginator .next a:visited { - font-weight: normal; + font-weight: normal; } .paginator .prev, .paginator .prev-na { margin-right:.5em; } .paginator .next, .paginator .next-na { margin-left:.5em; } .paginator .page a, .paginator .page a:visited, .paginator .curr { - font-weight: normal; + font-weight: normal; padding: 6px 12px; } .paginator .curr { - background-color: #eee; + background-color: #eee; border-radius: 4px; - color: #999; - font-weight:normal; + color: #999; + font-weight:normal; } .thumbnail { - min-height: 200px; + min-height: 200px; } /* patch view */ table.patchmeta th { - text-align: left; + text-align: left; } table.patchmeta tr th, table.patchmeta tr td { - text-align: left; - padding: 1px 10px; - vertical-align: middle; + text-align: left; + padding: 1px 10px; + vertical-align: middle; } .patchrelations ul { - list-style-type: none; - padding: 0; - margin: 0; + list-style-type: none; + padding: 0; + margin: 0; } .patchnav { - padding-left: 1em; - padding-top: 1em; + padding-left: 1em; + padding-top: 1em; } /* checks forms */ /* TODO(stephenfin): Merge this with 'div.patchform' rules */ .checks { - border: 1px solid gray; - margin: 0.5em 1em; + border: 1px solid gray; + margin: 0.5em 1em; } .checks th { - margin-top: 0em; - margin-left: -0.6em; - margin-right: -0.6em; - padding: 0.3em 0.3em 0.3em 0.6em; - background-color: #0A0A47; - color: white; - font-size: 100%; - font-weight: normal; + margin-top: 0em; + margin-left: -0.6em; + margin-right: -0.6em; + padding: 0.3em 0.3em 0.3em 0.6em; + background-color: #0A0A47; + color: white; + font-size: 100%; + font-weight: normal; } .checks td { - border-top: 1px solid gray; - padding: 10px 15px; - padding-left: 0.2em; - margin-top: 0em; + border-top: 1px solid gray; + padding: 10px 15px; + padding-left: 0.2em; + margin-top: 0em; } .checks td a { - text-decoration: none; + text-decoration: none; } .checks td a:visited { - color: #786FB4; + color: #786FB4; } .checks a:hover { - text-decoration: underline; + text-decoration: underline; } .patchlistchecks { - display: inline-block; - border-radius: 7px; - min-width: 0.9em; - padding: 0 2px; - text-align: center; + display: inline-block; + border-radius: 7px; + min-width: 0.9em; + padding: 0 2px; + text-align: center; } .patchlistchecks.success { - background-color: #82ca9d; + background-color: #82ca9d; } .patchlistchecks.warning { - background-color: #ffc95e; + background-color: #ffc95e; } .patchlistchecks.fail { - background-color: #ff5555; + background-color: #ff5555; } .checks .state { - font-weight: bold; - color: #ddd; + font-weight: bold; + color: #ddd; } .checks .state.success { - color: #82ca9d; + color: #82ca9d; } .checks .state.warning { - color: #ffe59a; + color: #ffe59a; } .checks .state.fail { - color: #f7977a; + color: #f7977a; } .comment .meta { - background: #f0f0f0; - padding: 0.3em 0.5em; + background: #f0f0f0; + padding: 0.3em 0.5em; } .comment .content { - border: 0; + border: 0; } .patch .content { - padding: 1em; + padding: 1em; } .patch-pull-url { - font-family: "DejaVu Sans Mono", fixed; + font-family: "DejaVu Sans Mono", fixed; } .quote { - color: #007f00; + color: #007f00; } -span.p_header { color: #2e8b57; font-weight: bold; } -span.p_chunk { color: #a52a2a; font-weight: bold; } -span.p_context { color: #a020f0; } -span.p_add { color: #008b8b; } -span.p_del { color: #6a5acd; } -span.p_mod { color: #0000ff; } +span.p_header { color: #2e8b57; font-weight: bold; } +span.p_chunk { color: #a52a2a; font-weight: bold; } +span.p_context { color: #a020f0; } +span.p_add { color: #008b8b; } +span.p_del { color: #6a5acd; } +span.p_mod { color: #0000ff; } .acked-by { - color: #2d4566; + color: #2d4566; } .nacked-by { - color: #2d4566; + color: #2d4566; } .signed-off-by { - color: #672d45; - font-weight: bold; + color: #672d45; + font-weight: bold; } .from { - font-weight: bold; + font-weight: bold; } /* bundles */ table.bundlelist { - margin-top: 2em; - margin-bottom: 4em; - margin-left: auto; - margin-right: auto; - border: thin solid black; + margin-top: 2em; + margin-bottom: 4em; + margin-left: auto; + margin-right: auto; + border: thin solid black; } table.bundlelist th { - padding-left: 2em; - padding-right: 2em; - background: #eeeeee; - border-bottom: thin solid black; + padding-left: 2em; + padding-right: 2em; + background: #eeeeee; + border-bottom: thin solid black; } table.bundlelist td { - padding-left: 2em; - padding-right: 2em; + padding-left: 2em; + padding-right: 2em; } /* forms that appear for a patch */ div.patchform { - border: thin solid #080808; - padding-left: 0.6em; - padding-right: 0.6em; - float: left; - margin: 0.5em 5em 0.5em 10px; + border: thin solid #080808; + padding-left: 0.6em; + padding-right: 0.6em; + float: left; + margin: 0.5em 5em 0.5em 10px; } div.patchform h3 { - margin-top: 0em; - margin-left: -0.6em; - margin-right: -0.6em; - padding: 0.3em 0.3em 0.3em 0.6em; - background-color: #222; - color: #999; - font-size: 100%; + margin-top: 0em; + margin-left: -0.6em; + margin-right: -0.6em; + padding: 0.3em 0.3em 0.3em 0.6em; + background-color: #222; + color: #999; + font-size: 100%; } div.patchform ul { - list-style-type: none; - padding-left: 0.2em; - margin-top: 0em; + list-style-type: none; + padding-left: 0.2em; + margin-top: 0em; } /* forms */ @@ -376,79 +376,79 @@ table.form { } span.help_text { - font-size: 80%; + font-size: 80%; } table.form td { - padding: 0.6em; - vertical-align: top; + padding: 0.6em; + vertical-align: top; } table.form th.headerrow { - background: #222; - color: #999; - font-weight: bold; - text-align: center; + background: #222; + color: #999; + font-weight: bold; + text-align: center; } table.form th { - text-align: left; - vertical-align: top; - padding-top: 0.6em; + text-align: left; + vertical-align: top; + padding-top: 0.6em; } table.form td.form-help { - font-size: smaller; - padding-bottom: 1em; - padding-top: 0em; + font-size: smaller; + padding-bottom: 1em; + padding-top: 0em; } table.form tr td.submitrow { - border-bottom: 0.2em solid #222; - color: #333; - text-align: center; + border-bottom: 0.2em solid #222; + color: #333; + text-align: center; } table.registerform, table.passwordform { - margin-left: auto; - margin-right: auto; + margin-left: auto; + margin-right: auto; } table.loginform { - margin-left: auto; - margin-right: auto; - width: 30em; + margin-left: auto; + margin-right: auto; + width: 30em; } /* form errors */ .errorlist { - color: red; - list-style-type: none; - padding-left: 0.2em; - margin: 0em; + color: red; + list-style-type: none; + padding-left: 0.2em; + margin: 0em; } .error { - color: red; + color: red; } /* generic table with header columns on the left */ table.horizontal { - border-collapse: collapse; - border: thin solid #e8e8e8; + border-collapse: collapse; + border: thin solid #e8e8e8; } table.horizontal th { - text-align: left; + text-align: left; } table.horizontal td, table.horizontal th { - padding: 0.5em 1em; - border: thin solid #e8e8e8; + padding: 0.5em 1em; + border: thin solid #e8e8e8; } /* generic table with header row */ table.vertical { - border-collapse: collapse; + border-collapse: collapse; } /* per suggestions here: https://github.com/twbs/bootstrap/issues/12238 */ @@ -458,53 +458,53 @@ table.vertical { .table-extra-condensed > thead > tr > td, .table-extra-condensed > tbody > tr > td, .table-extra-condensed > tfoot > tr > td { - padding: 1px 0.5em 1px 0.5em; - vertical-align: middle; + padding: 1px 0.5em 1px 0.5em; + vertical-align: middle; } table.vertical th { - background: #222; - color: #999; - font-weight: bold; - text-align: center; + background: #222; + color: #999; + font-weight: bold; + text-align: center; } table.vertical th, table.vertical td { - padding: 0.2em 0.8em; - border: thin solid #e8e8e8; + padding: 0.2em 0.8em; + border: thin solid #e8e8e8; } td.numberformat { - text-align: right; + text-align: right; } /* boxes */ div.box { - border: thin solid #080808; - margin: 1em; - padding: 0.5em; + border: thin solid #080808; + margin: 1em; + padding: 0.5em; } div.box h2 { - background: #222; - color: #999; - margin: -0.5em -0.5em 1em; -0.5em; - padding: 0.3em 0.3em 0.3em 0.6em; - font-size: 100%; + background: #222; + color: #999; + margin: -0.5em -0.5em 1em; -0.5em; + padding: 0.3em 0.3em 0.3em 0.6em; + font-size: 100%; } div.box table.vertical { - margin-left: auto; - margin-right: auto; + margin-left: auto; + margin-right: auto; } /* columns */ .leftcol { - float: left; - width: 49%; + float: left; + width: 49%; } .rightcol { - float: right; - width: 49%; + float: right; + width: 49%; } diff --git a/lib/packages/jquery/jquery.tablednd.js b/lib/packages/jquery/jquery.tablednd.js index 6cf3a850..f83c6d43 100644 --- a/lib/packages/jquery/jquery.tablednd.js +++ b/lib/packages/jquery/jquery.tablednd.js @@ -5,7 +5,7 @@ * Licensed like jQuery, see http://docs.jquery.com/License. * * Configuration options: - * + * * onDragStyle * This is the style that is assigned to the row during drag. There are limitations to the styles that can be * associated with a row (such as you can't assign a border—well you can, but it won't be @@ -28,14 +28,14 @@ * Pass a function that will be called when the user starts dragging. The function takes 2 parameters: the * table and the row which the user has started to drag. * onAllowDrop - * Pass a function that will be called as a row is over another row. If the function returns true, allow + * Pass a function that will be called as a row is over another row. If the function returns true, allow * dropping on that row, otherwise not. The function takes 2 parameters: the dragged row and the row under * the cursor. It returns a boolean: true allows the drop, false doesn't allow it. * scrollAmount * This is the number of pixels to scroll if the user moves the mouse cursor to the top or bottom of the * window. The page should automatically scroll up or down as appropriate (tested in IE6, IE7, Safari, FF2, * FF3 beta) - * + * * Other ways to control behaviour: * * Add class="nodrop" to any rows for which you don't want to allow dropping, and class="nodrag" to any rows @@ -47,7 +47,7 @@ * * Known problems: * - Auto-scoll has some problems with IE7 (it scrolls even when it shouldn't), work-around: set scrollAmount to 0 - * + * * Version 0.2: 2008-02-20 First public version * Version 0.3: 2008-02-07 Added onDragStart option * Made the scroll amount configurable (default is 5 as before) @@ -103,7 +103,7 @@ jQuery.tableDnD = { var rows = table.rows; //getElementsByTagName("tr") var config = table.tableDnDConfig; for (var i=0; i<rows.length; i++) { - // To make non-draggable rows, add the nodrag class (eg for Category and Header rows) + // To make non-draggable rows, add the nodrag class (eg for Category and Header rows) // inspired by John Tarr and Famic var nodrag = $(rows[i]).hasClass("nodrag"); if (! nodrag) { //There is no NoDnD attribute on rows I want to drag @@ -194,7 +194,7 @@ jQuery.tableDnD = { } } - + if (mousePos.y-yOffset < config.scrollAmount) { window.scrollBy(0, -config.scrollAmount); } else { diff --git a/manage.py b/manage.py index fe6189ee..e4266ade 100755 --- a/manage.py +++ b/manage.py @@ -3,8 +3,9 @@ import os import sys if __name__ == "__main__": - os.environ.setdefault("DJANGO_SETTINGS_MODULE", - "patchwork.settings.production") + os.environ.setdefault( + "DJANGO_SETTINGS_MODULE", + "patchwork.settings.production") from django.core.management import execute_from_command_line diff --git a/patchwork/tests/utils.py b/patchwork/tests/utils.py index 4ac9afe0..577183d0 100644 --- a/patchwork/tests/utils.py +++ b/patchwork/tests/utils.py @@ -23,8 +23,8 @@ from patchwork.models import SeriesReference from patchwork.models import State from patchwork.tests import TEST_PATCH_DIR -SAMPLE_DIFF = """--- /dev/null 2011-01-01 00:00:00.000000000 +0800 -+++ a 2011-01-01 00:00:00.000000000 +0800 +SAMPLE_DIFF = """--- /dev/null\t2011-01-01 00:00:00.000000000 +0800 ++++ a\t2011-01-01 00:00:00.000000000 +0800 @@ -0,0 +1 @@ +a """ diff --git a/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml b/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml index 48afac05..8b2a9c3d 100644 --- a/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml +++ b/releasenotes/notes/CVE-2019-13122-e9c63aa346ed15c2.yaml @@ -8,4 +8,4 @@ fixes: many mail systems, including Gmail, and be parsed and stored by Patchwork. When a user viewed a patch detail page for the patch with this message id, the script would be run. This is fixed by - properly escaping the field before it is rendered. \ No newline at end of file + properly escaping the field before it is rendered. diff --git a/tox.ini b/tox.ini index 0c03857e..617e73cd 100644 --- a/tox.ini +++ b/tox.ini @@ -44,7 +44,7 @@ commands = [testenv:pep8] basepython = python2.7 deps = flake8 -commands = flake8 {posargs} patchwork +commands = flake8 {posargs:patchwork manage.py} [flake8] # Some rules are ignored as their use makes the code more difficult to read:
An assortment of fixes identified through the integration of pre-commit. Signed-off-by: Stephen Finucane <stephen@that.guru> --- .dockerignore | 1 - htdocs/css/style.css | 382 +++++++++--------- lib/packages/jquery/jquery.tablednd.js | 12 +- manage.py | 5 +- patchwork/tests/utils.py | 4 +- .../CVE-2019-13122-e9c63aa346ed15c2.yaml | 2 +- tox.ini | 2 +- 7 files changed, 204 insertions(+), 204 deletions(-)