Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 23:18:39 +0000] rev 893
Added Ignored applications to the list view.
Also restored the New Club notification upon application acceptance. That broke in r1483.
Patch by: Lennard de Rijk
Reviewed by: to-be-reviewed
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 22:47:22 +0000] rev 892
Replaced boolean properties in soc/models/group_app with status property.
This property has 5 choices ['accepted','rejected','ignored','needs review','completed']
Patch by: Lennard de Rijk
Reviewed by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 22:21:40 +0000] rev 891
Move getToSLink() to soc.views.helper.redirects.getToSRedirect().
Fixes a circular import problem that prevented the site-wide ToS link from
being displayed by checkAgreesToSiteToS(). This is now completely implemented.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 21:41:36 +0000] rev 890
Tighten more checkIsUser() access to checkAgreesToSiteToS() instead.
Update the checkIsHost() comment.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 21:32:51 +0000] rev 889
Make checkAgreesToSiteToS() the default access check for many other checks.
Replace checkIsUser() with the more-strict checkAgreesToSiteToS(). Also, add
missing args and kwargs parameters to checkAgreesToSiteToS() that were missed
in the merge right before commit.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 21:25:18 +0000] rev 888
Add checkAgreesToSiteToS() (partially implemented until circular import fixed).
Need to fix a circular import problem before the alternate message will
include a link to the site-wide ToS Document.
Also, fixed the descriptions in the first few check...() functions.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 21:09:52 +0000] rev 887
Take advantage of new "from ... import ... as ..." rules in the style guide.
Patch by: Todd Larsen
Review by: to-be-reviewed
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 20:25:07 +0000] rev 886
Added clean_existing_user_not_equal_to_current to clean the backup Administrator field in an application.
Patch by: Lennard de Rijk
Reviewed by: to-be-reviewed
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 19:53:23 +0000] rev 885
Changed the redirect in club application overview to the correct page.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 19:45:13 +0000] rev 884
Changed checkIsClubAppAccepted into a more generic checkIsApplicationAccepted.
Also fixed a minor bug in club_app view.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 19:40:39 +0000] rev 883
Changed the collection name and help text for Organization member template.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 19:32:00 +0000] rev 882
Added club_app model and logic.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 18:50:22 +0000] rev 881
Added created and last modified timestamps to the group_app model.
Also edited the templates to show some extra information. The review template has the names for the new properties hard coded because the auto_add and auto_add_now datetimeproperties don't show up in the fields of the entity.
Patch by: Lennard de Rijk
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 18:25:37 +0000] rev 880
Fixed typo in r1446.
Patch by: Todd Larsen
Review by: to-be-reviewed
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 16:13:51 +0000] rev 879
Changes in Role model.
Clarified that we are asking for a shipping adress. Also moved Agrees to ToS to the bottom.
Patch by: Lennard de Rijk
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:13:18 +0000] rev 878
Fix trailing whitespace and apply decorators
Patch by: Sverre Rabbelier
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:13:03 +0000] rev 877
Fix whitespace damage
Patch by: Sverre Rabbelier
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:12:48 +0000] rev 876
Make use of the new decorators in all applicable views
Patch by: Sverre Rabbelier
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:12:31 +0000] rev 875
Renamed showReviewOverview to review_overview and reviewOverview
Patch by: Sverre Rabbelier
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:12:16 +0000] rev 874
Fixed two trivial bugs
A "kwargs=kwargs" instead of **kwargs
Missing 'and entity' check
Patch by: Sverre Rabbelier
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:12:01 +0000] rev 873
Added a merge_params and check_access decorator
The merge_params decorator is very simple and takes self._params and
merges it with params. It's main purpose is to explicitly mark the
views in which it is intended that self._params is merged raw.
The check_access decorator factors out the access checking process
and passes args and kwargs to the access checking methods.
Patch by: Sverre Rabbelier
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 21 Jan 2009 16:11:47 +0000] rev 872
Prepare access to receive args and kwargs as argument
Also use two "magic values" when the sidebar is doing the access
checks, this might be useful later.
Patch by: Sverre Rabbelier
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 15:35:10 +0000] rev 871
Reverted r1457.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 15:29:58 +0000] rev 870
Changed link in club_app accepted notification.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 15:11:45 +0000] rev 869
Added a new view to help with application -> group process.
The applicant view is available only when there is an accepted application present. This will be checked by the access checker in an upcoming patch. The view uses a special form so that certain fields cannot be edited by the user. These fields correspond to important fields filled in during the application process.
Create access for a club has now been set to Host level.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 14:57:12 +0000] rev 868
onCreate for a Club now marks any application for the same club as completed and invites the admins.
This is useful for when the club has been created without using the application.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 14:36:44 +0000] rev 867
Edited the method used to clean link_id in params.py.
Removed the clean_new_link_id from the logic/cleaning.py module because this is not used anymore.
This method should be replaced by a method that is used after all the fields have been cleaned so that the actual key_name of an entity can be constructed.
Patch by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Wed, 21 Jan 2009 13:55:06 +0000] rev 866
Fixed a typo in the comments of soc/views/helper/forms.py
Patch by: Lennard de Rijk
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 02:39:29 +0000] rev 865
Fix missing import identified by pylint.
************* Module app.soc.logic.models.user
E0602: 99:Logic.agreesToSiteToS: Undefined variable 'db'
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Wed, 21 Jan 2009 02:37:43 +0000] rev 864
Fix missing import identified by pylint.
************* Module app.soc.logic.models.presence
E0602: 56:Logic.getToS: Undefined variable 'db'
Patch by: Todd Larsen
Review by: to-be-reviewed