summaryrefslogtreecommitdiffstats
path: root/apps/patchwork
Commit message (Collapse)AuthorAgeFilesLines
* views: separate project views into separate fileJeremy Kerr2009-04-053-15/+40
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* tests: test creating bundles with existing namesJeremy Kerr2009-02-271-0/+14
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Add extra patch meta tags (Tested/Reviewed/Nacked-by)Mauro Carvalho Chehab2009-02-152-1/+3
| | | | | Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Fix non-ascii character encodings on xmlrpc interfaceMauro Carvalho Chehab2009-02-151-6/+6
| | | | | | | | | | The current xmlrpc interface dies when trying to retrieve patches that contain non-ascii characters. This change uses unicode() rather than str(). Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Check count() for duplicate bundle patches, rather than try/exceptJeremy Kerr2009-02-132-4/+49
| | | | | | | If the exception is raised, the transaction will still be in a bad state, due to the foreign key constraint. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Don't create bundles with no nameJeremy Kerr2009-02-132-1/+27
| | | | | | Check to see that we have a valid name before creating. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [mbox] Add extra newline after patch commentJeremy Kerr2009-02-121-1/+1
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [mbox] Put ACKs before '---' updatesJeremy Kerr2009-02-123-2/+94
| | | | | | | | | Update the Patch's mbox() function to split into '--- update' sections, allowing us to add Acks and Signoffs in the right place Add tests too. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [tests] Add initial bundle testsJeremy Kerr2009-02-103-3/+317
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Bundle reordering supportJeremy Kerr2009-02-084-28/+61
| | | | | | | | Bundles can now be reordered and saved. Add dependency on jquery in INSTALL. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Allow Order class to be user-editableJeremy Kerr2009-02-082-5/+11
| | | | | | | This can be enabled with the editable_order parameter to generic_list Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Initial bundle reordering supportJeremy Kerr2009-01-291-1/+17
| | | | | | | | | todo: * implement ajax reorder api * finish migration script * order patches in bundle view Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [models] use case-insensitve lookup when creating confirmationJeremy Kerr2009-01-271-1/+1
| | | | | | | We were getting multiple entries when the case-sensitive lookup failed. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Use 'from' import for urlresolversJeremy Kerr2009-01-071-3/+2
| | | | | | | Some python installs can't seem to find the fully-qualified module, so use the 'from'-style import instead. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [models] Make patches unique on (msgid, project), not just (msgid)Jeremy Kerr2008-10-293-10/+80
| | | | | | | | | | On patchwork.ozlabs.org, we may see multiple patches for different projects, but with the same message-id. We want these patches to show up on both projects, so we need to change the current UNIQUE contstraint on msgid. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Use standard UserAdmin for admin siteJeremy Kerr2008-10-231-2/+1
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Handle special case of "New newline at end of file"Nate Case2008-10-231-0/+3
| | | | | | | | | | | | | | | If a file doesn't have a newline at the end, diff will insert a "\ No newline at end of file" line in the middle of the last hunk. parser.py was counting this line as part of the hunk line count. This had the side effect of putting the last line of the hunk in the patch comment, and truncating out the last line of the diff. Handle this special case by not including this line in the internal line counter. This appears to be the only case where diff will insert a string like this within a hunk. Signed-off-by: Nate Case <ncase@xes-inc.com> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Handle patches with no content charset definedJeremy Kerr2008-10-232-2/+19
| | | | | | If we don't have an incoming charset defined, assume utf-8. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Convert patch content to unicode before parsingJeremy Kerr2008-10-141-0/+2
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [packages] Fix bug with django 1.0Jeremy Kerr2008-10-101-0/+1
| | | | | | We needed an extra to_field_name Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Accept x-diff patchesJeremy Kerr2008-10-102-3/+8
| | | | | | We should accept x-diff attachments as well as x-patch. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [packages] update to django 1.0Jeremy Kerr2008-10-103-2/+2
| | | | | | With a minor code change, and a small workaround. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [models] Fix exception on mbox view with non-ascii submitter nameJeremy Kerr2008-10-092-2/+25
| | | | | | | | We need to unicode the name instead of str-ing it. Add test. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [encoding] Don't output patch mbox as quoted-printableJeremy Kerr2008-10-072-3/+21
| | | | | | | | git-am doesn't like quoted-printable, so output mbox files as raw 7- or 8-bit mbox files. This means we have to create a new MIMEText class, so that the content isn't automatically QP-encoded on __init__(). Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [models] Create patch mbox with utf-8 encodingJeremy Kerr2008-09-293-3/+59
| | | | | | | | | We were getting exceptions on the mbox view when looking at a non-ascii patch. Add test to suit. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [sql] Default data should be in fixtures, not custom SQLJeremy Kerr2008-09-294-26/+73
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [tests] Remove print from SenderCorrelationTestJeremy Kerr2008-09-291-2/+0
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [tests] Add tests for utf-8 patchesJeremy Kerr2008-09-294-14/+35
| | | | | | .. which expose a bug in the patch parser, so fix that. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Don't use bundle owner for current userJeremy Kerr2008-09-251-2/+2
| | | | | | | | | This made the top auth area show the bundle owner's details, not the bundle owner's. Bundle owner can still be accessed through bundle.owner in the template. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Change default ordering to latest-firstJeremy Kerr2008-09-241-3/+3
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Restructure profile view, simplify bundle accessJeremy Kerr2008-09-233-1/+31
| | | | | | | Make bundles more like todo lists - the list itself has its own page, accessible from the top user links. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [views] Fix empty bundle showing entire project's patchesJeremy Kerr2008-09-201-1/+1
| | | | | | | | | | If a bundle is empty, we end up triggering the 'if not patches' check in generic_list(), which repopulates the patch list with the current project's patches. So, check for None instead, and leave the patch list empty. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Add comment signoffs and acks to mbox textJeremy Kerr2008-09-201-1/+16
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [test] Move helper functions into tests/utils.pyJeremy Kerr2008-09-202-40/+83
| | | | | | So that we can use them for other tests Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [models] Improve comment filterJeremy Kerr2008-09-201-1/+1
| | | | | | We should be able to search on the patch_id faster. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Fix spacing for encoded headersJeremy Kerr2008-09-202-9/+13
| | | | | | We need to join the fragments with a space. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Merge senders with different caseJeremy Kerr2008-09-202-1/+42
| | | | | | ... and add tests Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [test] 2.4-ise test casesJeremy Kerr2008-09-201-2/+8
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Add bin/__init__.pyJeremy Kerr2008-09-201-0/+0
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Decode From: headersJeremy Kerr2008-09-202-5/+59
| | | | | | | | | We're getting a few utf-8 encoded From: addresses, so decode before saving to the DB. Also, add tests. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [parser] Don't remove --- update linesJeremy Kerr2008-09-202-1/+26
| | | | | | | | | We'd like to keep update lines in the patch comments, so change the signature-removal code to allow them to pass through. Also, add appropriate tests. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [test] Add test for signatures in commentsJeremy Kerr2008-09-201-0/+13
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* [test] Add patchwork testing infrastructureJeremy Kerr2008-09-193-0/+131
| | | | | | Add a small set of initial tests for the patch parsing code. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Remove superfluous print from parsemail.pyJeremy Kerr2008-09-191-1/+0
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Fix handling of newlines in subjectsJeremy Kerr2008-09-161-0/+5
| | | | | | By doing two passes of whitespace normalisation Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Drop project.linkname from patch subject linesJeremy Kerr2008-09-161-3/+3
| | | | | | .. otherwise every cell patch will be prefixed with [Cbe-oss-dev] Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Improve [PATCH] subject parsingJeremy Kerr2008-09-161-4/+89
| | | | | | | We'd like to leave some fragments of [PATCH]-style headers in the patch subject, so add functionality to do this in clean_subject Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Fix mishandling of existing querystring parametersJeremy Kerr2008-09-121-1/+1
| | | | | | | Currently, we only get the first character of order= parameters in the 'remove this filter' querystring. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Remember state filter settingsJeremy Kerr2008-09-121-2/+12
| | | | | | We need to set a few 'selected' options.. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
* Don't apply delegate = Nobody filter by defaultJeremy Kerr2008-09-121-18/+11
| | | | Signed-off-by: Jeremy Kerr <jk@ozlabs.org>