Revert "Include patch id in commitfest page"

This reverts commit eee60a5332ac0745071086477ff4e189b080d3ab.

Discussion: https://postgr.es/m/ff9d7921-f4e3-498a-9dba-ddd28fdb359f%40eisentraut.org

Include patch id in commitfest page

Clicking the patch header would sort by the patch ID, but that ID would
not actually be visible. Effectively resulting in random ordering for
the user, so let's just show the ID.

Add note about pre-commit hook

Add missing setup.cfg

Seems I forgot to git add this ages ago :/

Spotted by Jelte Fennema-Nio

Update django in requirements.txt to be 4.2

We've been running 4.2 in production for a long time by now, but forgot
to update the requirements.txt file. Oops...

Add LICENSE file

We use the PostgreSQL license, lets make it explicit.

Spotted by Jelte Fennema-Nio

DOn't crash on very long annotations

When a very long annotation is posted, we would crash if it's shorter
than 500 characters, but the length of the *log* message became longer
than 500. To handle this, shorten the text in the log message if it's
too long.

Minor pycodestyle updates

Allow searching by email Message-ID

Redirect straight to patch if search finds only one

Add search bar to search page, so you can search again

Add instructions to checkout latest CFbot patchset

Add extra links to the patch page

Make homepage more useful

A long list of ancient commitfests isn't super useful. This puts a few
links at the top of the page that cover the most common usages of the
commitfest app.

Add login redirect for dev environments

Add a generic dev uwsgi script and update readme

Don't crash on invalid target versions

Add missing newline, breaking pycodestyle

Add a stable URL for a patch that redirects to latest cf

Links to patches get outdated ever time we move it to new commitfest.

Add /current link that links to either /inprogress or /open

Redirect /open fully

Now it forwards everything after the slash so that links like the
following work:

/open/new/
/open?author=-3

Update url matching for django 4.2

Update auth plugin with signal config for django 4.2

Sync up to latest version of pgweb auth plugin

Don't crash on missing parameter

Change static->media and stop using collectstatic

This changes both the storage location of the files and the URL. This
aligns it up with how (almost) all our other django apps work, so good
for consistency.

Http404 is an exception to raise, not a class to return

Restrict user search/import to cf admins

All users can still enumerate local users, but the functionality to
search the central database is restricted to admins only.

Reported by Benjamin Flesch

Switch out pep8 for pycodestyle in git hook

New name for the same command.

Update README and development requirements

Update for django 3.2 compatibility

Silence pep8 warnings

Replace django-selectable with selectize

django-selectable is no longer maintained, and will cause issues in
newer versions of django.

selectize is what we use in the pgeu-system codebase today, so copy the
handling over from there.

Properly allow moving of WoA patches to next CF

The previous commit removed the refusal, but missed adding the allwing..

Allow moving of Waiting On Author patches to next CF

Discussion: https://postgr.es/m/20211001173131.GA13090@ahch-to

Properly escape list of messages when creating a new entry

This has been broken since day 1... oops.

Noted by Vik Fearing

Remove unused import

Move handling of secondary email addresses upstream

Since the upstream main website now handles secondary email addresses,
centralize the handling to there. This removes the local handling
completely, except that we store them in the database. The new
push-changes API ensures that they are kept in sync with upstream.

Import latest version of community authentication plugin

This includes now accepting push changes from upstream.

Another django 2.2 update

Updates for django 2.2

Update to latest version of django-selectable

Set on_delete=models.CASCADE on all relations

Remove unused file

Update favicon to match the main website, per Peter Eisentraut.

Sync up community auth plugin to latest-and-greatest

Remove reundant role attribute

Don't include "target version" on new patch form

Remove trailing slashes from message-id URLs

This avoids useless redirects.

Change backgroud color of target version label

Per request

Another markup fix

Fix markup

Add index to patchhistory.date

Fix activity feed queries

These clearly had little to do with reality since they would return
duplicate entries if a patch was in more than one cf..

Add a field to each patch for target version

This is particularly interesting towards the end of a cycle where it can
be used to flag patches that are not intended for the current version
but still needs review.

Add a pre-commit hook to ensure pep8

Fix mail sender for py3

Update migrations for py3 unicode

Need to change old migrations not to indicate we need new ones.

Update cmd tools to use python3

This includes changing to requests for check_patches_in_archives.py,
like previously done for the internal APIs.

Remove oneoff tool instead of having to port it

Fix class property reference for py3

Unicode fixes for python 3

Import latest, py3 compatible, auth.py

Fix imports and exceptions for Python3

Fixes for pep8 compatibility

Convert all spaces to tabs

Switch to using requests for archives API

This should fix some unicode issues

Change sort order for reviewers/authors in filter dropdown

Previously it was sorted by last name, which did not always make sense.
Canged to sort by first name, which hopefully makes more sense.

Per request from Alvaro Herrera and others.

Add "withdrawn" commitfest status

By popular (?) request

Merge old model changes that never made it into migrations

Clarify some buttons

Add some longer texts to indicate what's really hiding behind buttons.

Don't use hardcoded pgsql-hackers name in status message

Instead use the value from the settings file, which is what's actually
used to send.

Include DKIM/SPF instructions on commit/review forms

Clarify what subscribe/unsubscribe button does

Also don't fail on invalid reviewers or authors

Same fix as the previous one for status, but applies to the other two
filter-by-dropdown fields.

Don't fail on incorrect statuses in URL

The comment said we should ignore the status if int() on it fails, but
to do that we have to fail it before we actually add the WHERE clause.
Thus, reorder the code to do so.

Fix colspan

Bring back topic listing

Accidentally broken when query was rewritten

Rewrite main CF dashboard query to use SQL

We've long gone past what the django ORM can handle and got into
hundreds of queries for large commitfests. And it's not really that hard
to rewrite as SQL, even though we have to dynamically build the WHERE
clauses. So do that.

Show the number of CFs a patch has been on

Per discussions at the developer meeting, show the number of CFs a patch
has been on, and make it possible to sort on this value.

Don't crash when passing in invalid values for filters

Previously we'd throw a 500 internal server error, and cause error
logging to happen. This became evident when scripts trying to SQL-inject
the site started hitting us..

Instead, just ignore any filters that pass non-integer values into
integer fields completely.

Fix template loaders for django 1.11

Seems django 1.11 automatically enables caching template loader, which
of course breaks the ability to make any changes to the pages of a
website without restarting it. And there is no way to turn it off other
than to explicitly configure individual loders (the logic to turn it on
in non-debug configurations is hardcoded and cannot be changed).

Fix model diffing for django 1.11

Don't allow null wikilink and gitlink

Both had a default set to '', so also put that in the model. Needed to
give correct diffs in newer djangos

Upgrade django-selectable to v1.1.0

Needed for newer django

Fix warning triggered in django 1.11

Implement thread notification receiving

This allows the archivs server to ping the CF app to pick up updates to
mailthreads quicker.

Update format of urls.py to be compatible with newer django

Replace render_to_response with render

This is rquired as part of the move to a newer Django, and works fine on
older versions too

Fix template comparison operators

Update template configuration syntax

Change to syntax compatible with newer Django, still works on current
version.

Actually deliver CCed emails to patch authors

Previously we would set the Cc header of the email, but not actually
send a copy to the user. That way if the user was not subscribed to the
hackers email, they'd end up not receiving the email at all, which
somewhat defeats one of the original points of adding the Cc in the
first place.

Spotted by Peter Geoghegan

Update README file

While at it, convert to Markdown format for nicer rendering.

Fix typo

Hide annotation button if user is not logged in

Reported by Peter Geoghegan

Fix typo

Fix closing of <table> tag

Fix incorrect reference to global templates directory

This has probably been needed since an upgrade a long time ago but got
stuck in a separate branch. Fixes the case where the admin interface
could end up rendering a login form that would then not work, instead of
providing the proper redirect.

Remove deprecated and superfluous <script> attributes

The language attribute has been deprecated for quite some time with
the type attribute setting the type of embedded content.  The type
attribute has further been defined to text/javascript as the default
value in the spec, so no attribute should be used for Javascript.
Per the standard: https://www.w3.org/TR/html5/scripting-1.html

Fix broken markup in future commitfests