tox: Cleanup '.pyc' files before run

Sometimes leftover '.pyc' files can cause tests to fail. Ensure this
never happens by always removing these files on start.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

models: Rename 'Check.__unicode__' to '__str__'

This causes issues with Python 3.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

models: Resolve SubfieldBase deprecation warnings

SubfieldBase has been deprecated. It is necessary to add a
'from_db_value' function in place of this mixin.

https://docs.djangoproject.com/en/1.9/releases/1.8/#subfieldbase

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

settings: Place most-used path first

Make the settings file a little easier to parse by placing the most
used (i.e. the one used with recent versions of Django) first.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

settings: Add TEMPLATES setting

This was added in Django 1.8 and will be required by Django 1.10.

https://docs.djangoproject.com/en/1.9/ref/templates/upgrading/#the-templates-settings

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

urls: Remove reverse by Python path urls

Reversing by Python paths has been deprecated in Django 1.8, causes
warnings in Django 1.9 and will be removed in Django 2.0. Resolve the
warnings and prevent issues in the future by importing by module
instead.

https://docs.djangoproject.com/en/1.9/releases/1.8/#passing-a-dotted-path-to-reverse-and-url

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Use URL names in place of Python paths

Reversing by Python paths has been deprecated in Django 1.8, causes
warnings in Django 1.9 and will be removed in Django 2.0. Resolve the
warnings and prevent issues in the future by referring to URLs by name,
rather than by Python path.

https://docs.djangoproject.com/en/1.9/releases/1.8/#passing-a-dotted-path-to-reverse-and-url

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

urls: Add url names

Reverse by name is an alternative to reverse by Python path. The names
must be added as a prerequisite for removing reverse by Python path,
per Django documentation.

https://docs.djangoproject.com/en/1.9/ref/urlresolvers/#django.core.urlresolvers.reverse

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Move view-related code from utils->views/__init__

These functions are view-related and belong in the views folder.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

views: Split up 'base'

This is a collection of unrelated functions, many of which belong in
their own file. Do this, taking the opportunity to rename some of these
functions to better indicate their purpose in the process.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Cleanup file

Remove some unnecessary code and reorder in a logical fashion

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Add django19 targets

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: 'dict_keys' object has no attribute 'sort'

Resolve an issue causing the aforementioned issue.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

CHANGELOG: Add autodelegation and Python3 features

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Report if a patch is delegated and to whom

When a patch is delegated, it can be important to report it via
pwclient, as handing delegated patches may be different. So, add
an optional field at the email's body if this is happening.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>
Suggested-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

parsemail: Generate useful retcodes

When things are broken at parsemail, we need to be able to
debug it. So, add some messages to it, in order to allow
checking what it actually did, and to let it return 1 if
an email got skipped by parsemail.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

forms: Allow the delegate field to keep its current value

When a patch is delegated at parse time (either through the
X-Patchwork-Hint mail header or through delegation rules), the delegate
might not be in the list of project maintainers.

Add the current delegate to the list of acceptable values for the
delegate field to allow the current value to be kept when editing the
patch.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

parser: Set the delegate using Delegation rules

v2: Resolve issues with unit tests

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

parser: Add patch_get_filenames()

The function returns the list of files touched by a patch, after
stripping the leading component of the path name.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

models: Add DelegationRule object

Delegation rules are used to automatically set the delegate of a patch
based on the files touched by the patch.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

tox: 'flake' -> 'flake8'

The 'flake8' target specified the wrong dependendency. Resolve this.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

parsearchive: Handle duplicates

The parsearchive tool can be used to load missing messages sourced
from mailman or another source. In this use case, there's a good
possibility that at least some of the messages found in the archive
are already stored in patchwork. Handle this case by ignoring these
duplicates.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

parsearchive: Fix import bug

A bug was introduced in the Python 3 port - the 'bin' folder is not a
module and therefore relative imports should not be used. Resolve this.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

parsemail: Parse Mailman-formatted addresses

If loading archives from mailman, it is useful to parse and format
addresses such that they are proper emails.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

pep8: Manually resolve remaining issues

The 'autopep8' tool can't do everything, and it is necessary to resolve
some final issues. Most of these issues fall under the following
categories:

  E501  line too long
  E241  multiple spaces after ','
  F401  'module' imported but unused
  F841  local variable 'name' is assigned to but never used

It is also necessary to insert '# noqa' comments to hide some F403
errors ('unable to detect undefined names') where 'import *' is useful.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

pep8: Autoresolve most PEP8 issues

...using the 'autopep8' tool.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Remove ignored PEP8 errors and directories

Remove the following 'ignore' errors:

  E121-128,131  various visual indentation errors
  E251          spaces around keyword / parameter equals
  H405          multi-line docstring must have summary line

The only error remaining is E129, which is technically not part of the
PEP8 standard and contributes in practice besides lots of manual line
wrapping.

In addition, there's no good reason why the tests directory shouldn't
be assessed like everything else - tests are still code and, as such,
should be held to the same standards..

Finally, include the 'pwclient' script which should not be ignored.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: Remove all deprecated UnitTest calls

Remove the likes of 'failUnlessEqual' and 'assertEquals', which have
been deprecated and cause warnings in Django 1.6 with Python 3.4.

https://docs.python.org/2/library/unittest.html#deprecated-aliases

This allows us to remove an item from the TODO list.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: Resolve unicode issues

Python 3 is unicode only. While many of the issues with unicode, such
as the now invalid 'u' prefix, have already been resolved, there are a
few more issues.

Many of these issues are related to HTTPResponse.content, which returns
bytes and needs to be "decoded" in order to perform actions like
concatenation with str objects (unicode). Where possible, make use of
assertContains, per the Django documentation (http://bit.ly/1lRDYie),
else fall back to including a 'decode' statement.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: max does not accept 'None'

max(n, None), where n is a valid number, is valid in Python 2 but not
Python 3. Replace these calls with something that works in both
versions of Python.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: Fix Django models Python3 compatibility

Per suggestions from the Django porting guide:

    https://docs.djangoproject.com/en/1.8/topics/python3/

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: "Modernize" code base

Run code through the 'modernize' application to fix Python 3
compatibility while also retaining Python 2 backwards compatibility.

There are some key changes made to the autogenerated code:

* Don't wrap 'items()' in 'list' for for loops - it's not necessary
* Don't wrap 'keys()' in 'list' - just drop 'keys()'
* Use Django's version of six rather than the upstream one

Many of the issues found are based upon the changed definitions of the
map, keys and items functions, along with the removal of the iteritems
function and reduce keyword.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

trivial: Reorder imports

Imports are easier to read if they are grouped by type (standard
library, third party, application) and organised alphabetically. Make
it so.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

py3: Add required 'future' imports

These are quite limited as patchwork only supports Python 2.6+. As
such, only the 'print_function' and 'absolute_import' statements are
required.

Found using 'modernize'

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

requirements: Update version of dateutil used

The version of dateutil currently used does not support Python3, so
instead use the latest version.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Add Python 3.4 targets

These don't currently pass, but they will allow for testing as Python
3 compatibility is added.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Fix bug in 'coverage' target

The '--noinput' argument for './manage.py test' was in the wrong place.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

requirements: Fix bug in 'requirements-dev.txt'

There was a circular dependency. Resolve this.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

bin/parsearchive: Add rudimentary archive parsing

Add simple parsing of mbox archive files, as downloaded from a Mailman
instance. Currently this does not report errors or provide statistics
of any form.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

bin/parsemail: Use logging instead of prints

This is more useful for helpful dialog messages. Without this it is
necessary to add prints to figure out why patches aren't parsed.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

bin/parsemail: Support user-provided list ID

Sometimes mails won't contain the headers necessary to extract a
mailing list's ID (for example: mails downloaded from Mailman
archives). However, should the user already know the correct mailing
list ID then this extraction is not necessary.

Allow the user to provide a mailing list ID by the command line.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

trivial: Cleanup of 'bin/parsemail'

Do some cleanup of the script, removing excess whitespace and imports,
localising variables where possible etc. This file still isn't py3
compatible but it's getting there.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

management: Create 'rehash' command

The rehash script, though undocumented and possibly unused at the
moment, likely has some value to some users. Howver, it makes more
sense to provide this command as a management command like 'retag'.
Do this.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

settings: Add console email backend

Requiring configuration of an email server and email recipients for
debugging of email-based features is impractical. Use the console
backend provided by Django as the development default.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

login: add link to password reset

Refurbished version from 2010

Doesn't mention the username in the mail but in the confirmation step.
Essentially the templates are all optional but they do look better (and
more integrated in patchwork) than the default django admin thing.

No testing beyond some bored clicking here and there.

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

Cleanup requirements

* python-dateutil is only used in tests, so remove it from the
  production requirements
* mysqlclient is prefered to MySQLdb, so use that [1]
* selenium does not have its version fixed, so resolve that

[1] https://docs.djangoproject.com/en/1.8/ref/databases/#mysql-db-api-drivers

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Document all the 'PW_' environment variables

These are found in multiple places across the code. It would be helpful
to document them for the user.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Rework configurable 'PW_TEST_DB_xxx' settings

There are a number of environment variables that users can set to
configure different aspects of their testing environment. Rework these
like so:

* People use PostgreSQL, so make it as easy as possible for them to
  develop and test against it. Add a 'PW_TEST_DB_TYPE' setting
* Attempt to use defaults for the username and password in settings
* Allow the user to configure the database name, if they so wish

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Add requirements-test.txt

It's annoying that Django is not installed by default when setting up
a development environment. This is currently necessary because tox
uses these requirements files and it needs to test against multiple
versions of Django. Resolve this issue by adding a 'requirements-test'
file, thus allowing for Django-ful and Django-less scenarios.

This also allows us to delete 'requirements-base', which was a good
idea but alas contained too few options to really be viable. A little
duplication is better here.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Move requirements into base directory

These are documentation and therefore don't belong in the documentation
directory. The base directory is the usual location for these files, so
move them there.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Save the test project

create_email() can use the default project for its mail. If that is the
case, chances are we are going to use that email later in a test where
we'll want that project to exist.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Make sure all emails have a valid msgid

Messages ids will be used by the Series code as the way to uniquely
identify series. We must ensure every single email that goes through the
parser has a valid msgid to not fail once we parse series.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Remove 'apps' from paths

The patchwork directory structure has moved away from putting the
Django app(s) into 'apps/'

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Fix a few code blocks

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviwed-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Remove unhelpful command

That doesn't provide much value and can be removed.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Fix a missing 3rd person 's'

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Remove duplicated 'the'

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

retag: Properly display the final count

i == count cannot happen in the loop as i will vary from 0 to count - 1.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

retag: Remove excess line endings

In:

    commit 544b8bbcc7ec80d94c96f181886c51b177530a95
    Author: Stephen Finucane <stephen.finucane@intel.com>
    Date:   Fri Aug 21 15:32:17 2015 +0100

        trivial: Resolve PEP8 issues with 'management'

I noted that it wasn't all trivial changes. And indeed using the
stdin/stdout wrappers change the intended behaviour by adding a new
line.

The stderr wrapper also colors the line in red. Changed the last message
to be printed on stdout, seems more logical.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

.gitignore: patchwork/settings/production.py

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

TODO: cleanup deprecated unittest aliases

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

tests/test_user: Remove duplicate test

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Reviewed-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Don't use exceptions for flow control

Using exceptions for flow control is bad. Be consistent and instead
use proper functions to check for version support. This also allows
the use of tools to automatically identify feature flags when
removing support for Django versions in the future.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Revert "Remove Django < 1.7 blocks"

This reverts commit ea050ad02c61ff1f0bd03ffb02b4706817401aee.

To allow patchwork deployment on Enterpise versions of Linux, it is
necessary to continue to support Django 1.6.

Revert "tox: Remove the '-django16' targets"

This reverts commit 05bedc9b7678901fd6c590f32c217e1992bd290f.

To allow patchwork deployment on Enterpise versions of Linux, it is
necessary to continue to support Django 1.6.

Revert "docs: Note removed support for Django 1.6"

This reverts commit c9edb41d363b0c1926c530e8dc4c39a75e615397.

To allow patchwork deployment on Enterpise versions of Linux, it is
necessary to continue to support Django 1.6.

pwclient: Integrate 'checks' endpoint

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

views/xmlrpc: Add 'checks' endpoint

This includes adding a serializer plus the endpoint itself.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>
--

v2: Version now returns a tuple to allow for sematic versioning

templates/patch: Add check summary panel

Add a table to display the checks associated with a patch. This
includes the requisite styling along with some additional filters.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>
--

v3: Slight restyling per web UI rework

templates/patch-list: Add patch "checks" column

Add a column to display the important "checks" fields for each patch.
Note that only the "completed" checks are shown (i.e. "in progress" and
"not started" checks are ignored).

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

models: Add properties related to checks

Add properties for 'Patch' to get the unique checks associated with
a patch, the total number of each type of check and the combined state
of the check. These will be necessary to display this information to
the user.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

models: Add 'check' model

This will represent the status of tests executed (or executing) against
a patch. This includes a suitable migration and admin view.

Signed-off-by: Stephen Finucane <stephenfinucane@hotmail.com>
--

v3: Remove manual SQL migrations - they're not necessary/possible for
  Django 1.7/1.8, respectively

trivial: Cleanup of 'admin.py'

Resolve some issues with the aforementioned file. These are mostly
stylistic changes.

In addition, add a missing license. The date and author are
calculated from Git logs.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

trivial: Cleanup of 'models.py'

Resolve some issues with the aforementioned file. These are mostly
stylistic changes.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

Remove Django < 1.7 blocks

Seeing as we no longer support these versions of Django, there is no
need to keep these blocks around.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Remove the '-django16' targets

This version of Django 1.6 is no longer supported and there is no need
to test against it.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Note removed support for Django 1.6

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

docs: Emphasise the deprecation of Django 1.6

Though already mentioned in the CHANGELOG, the UPGRADING document
should describe plans to deprecate support for Django 1.6. This will
ensure sysadmins are not suprised when they attempt to upgrade.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

CHANGELOG: Add XML-RPC changes

This was missed in the original patchset.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

CHANGELOG: Add UI rework and Selenium features

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Add a way for the user to skip selenium tests

It's a bit more convoluted to setup the test environment to work with
selenium (eg. needs a browser installed and Xorg running, ...). It's
possible someone would want to skip those.

v2: Use 'PW' prefix rather than 'PATCHWORK', per existing test
  parameters, and pass said variable through tox

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Add a couple of Selenium tests

While developing the new series UI, several bug crept in but weren't
discovered until later. All because we don't have in-browser tests to go
along the lower level tests we already have.

In particular, behaviours that need javascript to run cannot be tested
outside of a full environment with the pages being served to an actual
browser.

This commit introduces selenium to the test suite and starts with 2
simple tests to give a taste of what it looks like.

test_default_focus: make sure we do focus the username field on the
                    login page

test_login: shows how to chain actions to test the full login phase.
            This is quite similar the lower level test, except it also
            checks we display the username once logged in.

v2: Use LiveServerTestCase for django pre-1.7
v3: Propagate the DISPLAY environment variable to have an X display
    specified for the browser
v4: Log execution of the chrome driver, useful for debugging
v5: Rebase on top of upstream
v6: Pass environmental variables to tox and ensure PEP8 compliance

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

tox: Use a range of ports for live servers

From:

    https://docs.djangoproject.com/en/1.8/topics/testing/tools/#liveservertestcase

In the case where the tests are run by multiple processes in parallel
(for example, in the context of several simultaneous continuous
integration builds), the processes will compete for the same address, and
therefore your tests might randomly fail with an "Address already in use"
error. For example:

    Traceback (most recent call last):
      File
      ".../python2.7/site-packages/django/test/testcases.py",
      line 1193, in setUpClass
          raise cls.server_thread.error
          error: [Errno 98] Address already in use

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Don't change settings.STATIC_URL in the XML-RPC tests

I tracked down a weird bug where the Selenium tests supposed to serve
all static files (.css, .js. ...) weren't doing so when running the
full test suite, but everything was working when running the tests in
isolation.

Turns out one of the other tests was changing settings.STATIC_URL
mid-way without restoring the original value in tearDown() and thus
impacting other tests.

It seems that this change wasn't necessary, the XML-RPC server doesn't
server any static file anyway.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

tests: Use --noinput when running tests

We don't want prompts when running tests, especially when they are run
through tox and most likely on a CI server.

When cancelling a test mid-way, the test db is not destroyed ans
subsequent runs will fail trying to create it and asking the user what
to do. --noinput will use the default answer, that is to destroy the
test db and start anew.

v2: Rebase on top of upstream
v3: Rebase, yet again.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com> (v2)

tests: Remove a spurious new line

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

filters: Fix submitter display when we only have an email address

I'm fairly sure I fixed that already, but here we go again. It's totally
valid to only have an email address for the submitter, so handle that
case properly.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

filters: Submit the form when <Enter> is pressed on a closed submitter field

<Enter> when an element of the form is focused should submit the form.
Before this patch, when <Enter> was pressed on the submitter button,
nothing would happen.

So, we tweak our <Enter> handling on the submitter input a bit more:
<Enter> can be used to select the submitter from the autocompletion
list, but, when pressed a second time (ie. when the autocompletion
option has be selected and the dropdown isn't shown), we submit the
form.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

filter: Tweak the autocompletion behaviour on <Enter>

No patch has landed yet and we've already had feedback from users (which
is a good thing!) that didn't want to lose the "quick" submitter search
by just typing a string and pressing <Enter> without having to wait for
the autocompletion to finish.

This will also make <Enter> do something useful for input strings that
have fewer than 4 characters, the autocompletion query only triggers
after that.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch-list: Use table-condensed to display the patch list

Wolfram didn't like the fact we're now seeing far fewer patches on a
single page. That sounds fair enough and the table-condensed bootstrap
class removes some padding, hopefully making it better.

Suggested-by: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch: Capitalize "Incoming Patches"

It's a title after all.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

base: Don't put the logout button in the is_staff block

Regular users didn't have their logout/profile drop down!

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch-list: Limit the number of chars to 100 for the patch subject

It can happen that people send a patch with a huge subject, by mistake.
Make sure it can interfere too much with the list of patches.

As an email subject can include extra words compared to a patch subject
(list header, tags, ...) let's have a slightly larger limit than 72 or
80.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch: Move the submitter and date out of the details

Trying to reduce the number of read-only fields in details.

Signed-off-by: Belén Barros Peña <belen.barros.pena@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

patch: Redesign slightly the comment headers

Signed-off-by: Belén Barros Pena <belen.barros.pena@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch: Put the download links next to the "Patch" header

v2: Fix 'dowload' typo x 2 (Bryce Harrington)

Signed-off-by: Belén Barros Peña <belen.barros.pena@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch: Override Bootstrap's <pre> styling

To look more like what patchwork was using.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>

patch: Pull the patch between the commit message and comments

This follows a logical flow, commit message, patch then comments.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

patch: Single out the commit message

All 'Comments' are stored the same way in the db, but I believe it's
worth making the distinction between introducing what the patch does and
eventual review comments.

v3: Fix docstrings
v2: Use two new Patch methods to retrieve the commit message and the
    other comments (called answers here) (Jeremy Kerr)

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>

patch: Remove the permalink item

One just has to copy the URL, no real need to duplicate that fact.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>