ajung/plone.app.discussion
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
1,670 Commits 1 Branch 0 Tags
ea99af48a2cfcd747d547a45de6d1fa39f2b6adc
Commit Graph
8 Commits
This Branch
This Branch
All Branches
Author SHA1 Message Date
Gil Forcada 2faf69bdbb Merge pull request #62 from plone/acquisition-total-comments-4 
Acquisition total comments
 2015-02-25 21:47:42 +01:00
Gil Forcada 2a423095ab Add note on CHANGES.rst 2015-02-25 21:47:08 +01:00
Gil Forcada 4ea41aba43 Fix tests 2015-02-16 16:35:43 +01:00
Gil Forcada d5b5b1c2cd Fix total_comments indexer 
"@property removes Acquisition wrappers from 'self'.
I don't remember why exactly (some implementation detail of Acquisition),
but it's a fact of life."

Quote from @davisagli on
https://github.com/plone/plone.app.discussion/pull/58

In short: removing the @property from total_comments fix the problem.
 2015-02-16 11:31:47 +01:00
Gil Forcada 535af3aace Count acquisition wrapped comments 
If the View permission is not set directly on the workflow,
but instead is left to be acquired,
total_comments will always return 0.
 2015-02-16 11:28:49 +01:00
Gil Forcada b4c1d9e42d Sort imports 
As per the styleguide
 2015-02-16 10:22:09 +01:00
Gil Forcada 19026439dd Use spaces please 2014-12-03 12:03:26 +01:00
Gil Forcada fa127ad85c Whitespaces cleanup 2014-08-23 00:31:28 +02:00