~dooferlad/patchwork/niceties : changes

~dooferlad/patchwork/niceties » Changes from revision 278

From Revision 269 to 250

Rev	Summary	Authors	Date
269	tests: Add tests for explicit delegate and state requests tests: Add tests for explicit delegate and state requests Signed-off-by: Dirk Wallenstein <halsmit@t-online.de> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Dirk Wallenstein	12 years ago
268	Use an explicit initial default patch state Use an explicit initial default patch state This will make editing states through the admin UI less error-prone, and will facilitate parsing patches when relying on a default state, as well as testing. Use the explicit default state when parsing mails. Signed-off-by: Dirk Wallenstein <halsmit@t-online.de> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Dirk Wallenstein	12 years ago
267	parser: allow words starting with "diff" at beginning of lin... parser: allow words starting with "diff" at beginning of line The patch parser splits emails incorrectly when the comment contains a line whose first word starts with "diff" (like "difficult" or "different"). For a real-life example of an email that triggers this bug, see http://patchwork.ozlabs.org/patch/135291/. Fix the issue by checking for a space after "diff". Add DiffWordInCommentTest to the test suite. Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Stefan Hajnoczi	12 years ago
266	sql: remove development migration script sql: remove development migration script The projectmaintainer heirachy hasn't been implemented, so we don't need the migration script yet. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
265	tests/patchparser: Add CVS format patch test tests/patchparser: Add CVS format patch test Test that CVS-formatted patches are parsed correctly, and that we capture the correct 'Index: ' metadata. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
264	tests: add abstract mbox parsing test tests: add abstract mbox parsing test Add a new class to handle mbox-parsing code, abstracting what is done by the current git-pull tests. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
263	filters: handle utf-8 characters when building filter querys... filters: handle utf-8 characters when building filter querystring Currently, we build the filter querystring by passing the raw string to urllib.quote, which expects an encoded string. When the raw string (in this case, a unicode object) contains a unicode character, quote() raises a KeyError. This fixes the problem by encoding the parameter name and values to UTF-8 first. Reported-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
262	patchwork: new pwclient git-am action patchwork: new pwclient git-am action This commit adds a new "pwclient git-am" action which applies given patch ID on the current git branch using "git am". It's convenient to keep track and authorship of patches applied locally from a patchwork server. Signed-off-by: David Decotigny <david.decotigny@google.com>	david decotigny	12 years ago
261	parser: allow other protocol types in pull-request handling parser: allow other protocol types in pull-request handling Change the parser's regex to allow other protocol types, not just git:// Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
260	models: Update Person & UserProfile object whenever User is ... models: Update Person & UserProfile object whenever User is saved Currently, UserProfiles are only saved on creation of the User object, before the first_name and last_name attributes are set. This means that we fallback to using the User.username value as the new Person.name. This change modifies User's post-save signal to always update the Person object, both on creation and updates. This means we get the proper name appearing on Person instances. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
259	Merge branch 'notifications' Merge branch 'notifications'	Jeremy Kerr	12 years ago
258	xmlrpc: do slice before patch_to_dict xmlrpc: do slice before patch_to_dict Currently, we map patch_to_dict before we slice the results (to only return max_count patches). This means that we hacve to retrieve all patches, then throw away most of the results of the map. This change does the slice on the patches before the map, letting django do a LIMIT-ed query instead. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	12 years ago
257	settings: Use multiple-DB-style format for default settings settings: Use multiple-DB-style format for default settings The DATABASE_* settings have been deprecated in django 1.3, replaced by the multiple-database style format (introduced in 1.2, which patchwork requires anyway). This change modifies the default settings.py to use the new format. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	13 years ago
256	Patchwork now requires Django 1.2, so state that in docs/INS... Patchwork now requires Django 1.2, so state that in docs/INSTALL Signed-off-by: Guilherme Salgado <guilherme.salgado@linaro.org> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Guilherme Salgado	13 years ago
255	Order: always set Order.order Order: always set Order.order If we create an Order objecty with editable = True, we don't initialise self.order, leading to testsuite failures with django-1.3. This change always initialises Order.order (and Order.reversed), so we don't exit the constructor without these initalised. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	13 years ago
254	forms: Fix MultipleBooleanField.to_python when the field val... forms: Fix MultipleBooleanField.to_python when the field value is missing If you write a test for, say, the bundle form of a patch list, you'd still have to specify the 'no change' value to other form (e.g. the multiple update one) fields using MultipleBooleanField or else it'd raise a ValueError when field.clean() is called as part of form._get_errors(). Signed-off-by: Guilherme Salgado <guilherme.salgado@linaro.org> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Guilherme Salgado	13 years ago
253	models: use implied line continuations models: use implied line continuations Use the implied line continuations, and make the indentation more clear. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Jeremy Kerr	13 years ago
252	models: Project.is_editable to check whether a user has edit... models: Project.is_editable to check whether a user has edit rights This is to replace the couple of places where we duplicate this same check. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Guilherme Salgado	13 years ago
251	views: Refactor generic_list() to make it less complicated views: Refactor generic_list() to make it less complicated When a form is submitted it now delegates to separate processing functions according to the action. Apart from being more readable it's now a lot easier to add extra forms for processing lists of patches. Signed-off-by: Guilherme Salgado <guilherme.salgado@linaro.org> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Guilherme Salgado	13 years ago
250	admin: Streamline admin model registration admin: Streamline admin model registration Use autodiscover and the new way to hook AdminSite into the URLconf. AdminSite.root() will be removed in Django-1.3 as noted here: http://docs.djangoproject.com/en/dev/internals/deprecation/ Signed-off-by: Dirk Wallenstein <halsmit@t-online.de> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>	Dirk Wallenstein	13 years ago

Older »