Sverre Rabbelier <srabbelier@gmail.com> [Sun, 02 Nov 2008 18:25:07 +0000] rev 431
Added a missing 'pass' statement
Sverre Rabbelier <srabbelier@gmail.com> [Sun, 02 Nov 2008 18:10:30 +0000] rev 430
Create a Site Settings sub-menu
Sverre Rabbelier <srabbelier@gmail.com> [Sun, 02 Nov 2008 17:26:06 +0000] rev 429
Fixed typo in e-mail address
Instead of 'sverre@rabbelier.nl' everything read 'sverer@rabbelier.nl'
Sverre Rabbelier <srabbelier@gmail.com> [Sun, 02 Nov 2008 17:25:10 +0000] rev 428
Fixed typo in access check
Instead of checking for 'public' the rights for 'edit' were checked.
Todd Larsen <tlarsen@google.com> [Mon, 27 Oct 2008 19:31:51 +0000] rev 427
The pretty blue triangles from Google Code!
Patch by: Todd Larsen
Todd Larsen <tlarsen@google.com> [Mon, 27 Oct 2008 19:19:44 +0000] rev 426
Add tooltips style display of help_text, instead of widening the form with
another (often ridiculously large) column. Also, version the .css and .js
files so that browsers won't display the wrong cached one when we change
something.
Patch by: Dmitri Gaskin, Todd Larsen
Review by: Todd Larsen
Todd Larsen <tlarsen@google.com> [Mon, 27 Oct 2008 18:54:53 +0000] rev 425
Change Site and User sub-menus so that the "header" entry for those sub-menus
is not a clickable link, but instead just a placeholder.
Patch by: Todd Larsen
Todd Larsen <tlarsen@google.com> [Mon, 27 Oct 2008 18:36:19 +0000] rev 424
Use nice "plus" and "minus" symbols from Google Code, instead of current
triangles, for sidebar menu.
Patch by: Todd Larsen
Augie Fackler <durin42@gmail.com> [Sun, 26 Oct 2008 21:36:29 +0000] rev 423
Add javascript expandable menus.
Patch by: Dmitri Gaskin
Review by: Augie Fackler
Augie Fackler <durin42@gmail.com> [Sun, 26 Oct 2008 21:26:35 +0000] rev 422
Add jQuery.
Patch by: Dmitri Gaskin
Review by: Augie Fackler
Augie Fackler <durin42@gmail.com> [Sun, 26 Oct 2008 00:57:33 +0000] rev 421
Remove bogus import from a previous iteration.
Sverre Rabbelier <srabbelier@gmail.com> [Sun, 26 Oct 2008 00:52:23 +0000] rev 420
Remove newlines in sidebar
Augie Fackler <durin42@gmail.com> [Sun, 26 Oct 2008 00:27:17 +0000] rev 419
Start a test system for the webapp itself.
Sverre Rabbelier <srabbelier@gmail.com> [Sun, 26 Oct 2008 00:13:26 +0000] rev 418
Always check if a user filled in a user profile
Also, if there is no checks defined, still do the general
checks.
Tim Ansell <mithro@gmail.com> [Sat, 25 Oct 2008 22:28:31 +0000] rev 417
Do not include attributes which are inherited.
Also some extra comments.
Review URL: http://codereviews.googleopensourceprograms.com/1603
Tim Ansell <mithro@gmail.com> [Sat, 25 Oct 2008 22:26:34 +0000] rev 416
Run with Python 2.5
Pawel Solyga <Pawel.Solyga@gmail.com> [Thu, 23 Oct 2008 12:00:29 +0000] rev 415
Fix typo in models/public.html template. Changes in Host Public Profile template (new header title and deletion of some fields).
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Thu, 23 Oct 2008 11:42:22 +0000] rev 414
Fix mistakes in Host List templates. Change Host List second row to Sponsor name instead of Sponsor link name.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Thu, 23 Oct 2008 11:20:35 +0000] rev 413
Rename Host list templates from group_<something> to host_<something> and move then to soc/host/list folder. Update affected files with new paths.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Thu, 23 Oct 2008 09:47:38 +0000] rev 412
Fix Sponsor profile edit/create template. Don't show "View Sponsor Public Profile" link and Delete button in Create Sponsor view.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Thu, 23 Oct 2008 09:08:02 +0000] rev 411
Addresses comments to r866.
http://code.google.com/p/soc/source/detail?r=866
Patch by: Pawel Solyga
Review by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Thu, 23 Oct 2008 05:21:41 +0000] rev 410
Moved all key_name related things to the logic modules
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Thu, 23 Oct 2008 05:21:26 +0000] rev 409
Addressed comments by Pawel and Todd
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Wed, 22 Oct 2008 17:18:57 +0000] rev 408
Inherit from BaseForm in all forms instead of from DbModelForm. This change doesn't include changes to templates and usage of as_table everywhere (work in progress).
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Wed, 22 Oct 2008 17:03:08 +0000] rev 407
Fix not working Create views (committed in r864). Check if entity exist in constructKeyNameSuffix funtion and apply changes to affected files.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Wed, 22 Oct 2008 16:55:35 +0000] rev 406
Fix some wrong paths to templates (committed in r864).
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Wed, 22 Oct 2008 16:21:50 +0000] rev 405
Add Google Analytics support to Site Settings. The reason I created additional SettingsValidationForm is because you cannot inherit from Form that has already defined Meta class, so it's sort of workaround for that. I didn't want to have same validation functions in both Form classes.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 22 Oct 2008 06:20:02 +0000] rev 404
Added a Host profile and generalized some views
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Wed, 22 Oct 2008 06:19:12 +0000] rev 403
Some more improvements to the generic view code
Namely getEmptyKeyFields and constructKeyNameSuffix,
some whitespace issues, and a few extra fields in the
context.
Patch by: Sverre Rabbelier
Reviewd by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Tue, 21 Oct 2008 01:22:36 +0000] rev 402
Incorperated suggestions by Todd
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Mon, 20 Oct 2008 23:24:41 +0000] rev 401
Make use of the new generic views
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Mon, 20 Oct 2008 23:24:32 +0000] rev 400
Added a generic version of the Document views
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Mon, 20 Oct 2008 23:24:21 +0000] rev 399
Various improvements to the generic View code
Among others:
* Added the 'page' argument to the simple.errorResponse call
* Pass the result of getEmptyKeyFields to edit() in create()
* Fix typo's in error responses
* Fixed a todo to properly redirect on create
* Made editPost return something when the form is not valid
* Added an _editGet hook
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Mon, 20 Oct 2008 23:24:07 +0000] rev 398
Rename 'existing_doc' and 'document' to entity
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Mon, 20 Oct 2008 23:23:53 +0000] rev 397
Remove the google search box and the google analytics page
These are left over from the mockups (removed because they
caused long wait times when no internet connection is
available).
Sverre Rabbelier <srabbelier@gmail.com> [Mon, 20 Oct 2008 23:23:43 +0000] rev 396
Improve the error message for getIfFields
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Mon, 20 Oct 2008 22:18:32 +0000] rev 395
Add documentation in response to Pawel's comments:
http://code.google.com/p/soc/source/detail?r=832
Patch by: Todd Larsen
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 21:12:08 +0000] rev 394
Add BaseForm class to soc.views.helper.forms module (work in progress). This changes the way as_table function displays the form (for more information have a look into doc string). BaseForm is going to be used for all forms in Melange in future. Right now it's still missing custom form errors labels and "required" text in 3rd column, but that's added as TODO and I'm working on it.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 21:08:08 +0000] rev 393
Add colon after label in _field_as_table_row template for consistency reasons with form.as_table output.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 18:00:14 +0000] rev 392
Add "View Sponsor Public Profile" to Sponsor Edit view template. Remove not needed <p> in home/public.html template. Add missing '$' at the end of some urls in site/map.py module.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 17:33:44 +0000] rev 391
Address comments to r844 and r845.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 16:17:32 +0000] rev 390
Remove unused imports, fix too long lines and indentions.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 13:14:40 +0000] rev 389
Fix identions, too long lines, unused imports and some other mistakes.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sun, 19 Oct 2008 13:06:20 +0000] rev 388
Add pylint configuration file (pylintrc) and do_pylint.sh script which runs pylint checkers on Melange code using pylintrc file as config. do_pylint.sh as default shows additional information like reports, TODOs, code similarities and unused imports, but you can run it in silent mode (--silent) which disables all of that. The only problem with unused imports in pylint right now is that it doesn't work in the situation described in last example at http://code.google.com/p/soc/wiki/PythonStyleGuide#Packages, so sometimes we get unused import soc when we actually shouldn't. However this can be fixed by writing pylint plugins (our own checkers) in future.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Sverre Rabbelier <srabbelier@gmail.com> [Sun, 19 Oct 2008 00:34:33 +0000] rev 387
Addressed comments by Todd on r822
Patch by: Sverre Rabbelier
Reviewed by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sat, 18 Oct 2008 21:55:38 +0000] rev 386
Fix identions in reflistprop module.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sat, 18 Oct 2008 21:54:13 +0000] rev 385
Remove not used imports in models. Fix missing spaces in models when operator not preceded by a space. Add missing ugettext_lazy import to Proposal model.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sat, 18 Oct 2008 17:49:46 +0000] rev 384
Add missing ugettext_lazy for string in soc.views.user.profile module.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sat, 18 Oct 2008 17:27:53 +0000] rev 383
Fix missing keyword argument in soc.views.user.profile.create() function when calling edit().
Patch by: Pawel Solyga
Review by: to-be-reviewed
Pawel Solyga <Pawel.Solyga@gmail.com> [Sat, 18 Oct 2008 17:25:04 +0000] rev 382
Change Sponsor delete url from /site/sponsor/profile/<link_name>/delete to /site/sponsor/profile/delete/<link_name> to match convention used in Document delete url.
Patch by: Pawel Solyga
Review by: to-be-reviewed
Chen Lunpeng <forever.clp@gmail.com> [Sat, 18 Oct 2008 05:59:04 +0000] rev 381
Fixed not working to edit a doc that selected from the docs list view. soc/templates/soc/docs/list/docs_row.html not full updated.
Patch by: Chen Lunpeng
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 05:30:57 +0000] rev 380
Change the "Site home page Document is missing" default text to give
instructions similar to those in the Site Settings form.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 05:26:53 +0000] rev 379
Remove another now-empty templates directory.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 05:25:47 +0000] rev 378
Nothing left in this templates directory, so remove it.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 05:11:37 +0000] rev 377
Refactor various site views into more generic locations, in preparation for
using access permissions to decide the fuctionality in the view, instead of
having lots of cut-and-paste copies of the same view functions.
site/home.py into more generic home.py
site/settings.py into more generic settings.py
site/docs/list.py into more generic docs/list.py
site/docs/edit.py into more generic docs/edit.py
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 04:56:39 +0000] rev 376
Add a style that appears in form error lists produced by Django Forms and
make it match the style of the existing td.formfielderror used by the
field_as_table_row helper.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 01:51:02 +0000] rev 375
Create package for soon-to-be-refactored group/home.py and group/settings.py
packages.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 01:47:02 +0000] rev 374
Add the Page object to the context of each view. Update templates to make
use of the newly-available page.long_name and page.short_name.
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 01:35:08 +0000] rev 373
Add the Page object to the context of each view method. Also, add some missing
Arg: sections to __doc__ strings and fix the missing context issue in
editPost() and editGet().
Patch by: Todd Larsen
Review by: to-be-reviewed
Todd Larsen <tlarsen@google.com> [Sat, 18 Oct 2008 01:32:40 +0000] rev 372
Formalize the concept of a NonPage that can appear in the site-map, useful for
non-clickable sidebar menu divisions, for example. Convert "hacky" fake Pages
into NonPages in the site-map.
Also, pass page=self to every view as a keyword argument when generating the
Django urlpatterns.
Patch by: Todd Larsen
Review by: to-be-reviewed