View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 2803 | Composr | points | public | 2016-08-28 06:43 | 2022-08-15 15:46 |
| Reporter | PDStig | Assigned To | PDStig | ||
| Priority | normal | Severity | minor | ||
| Status | resolved | Resolution | fixed | ||
| Summary | 2803: Unvalidating and then revalidating content results in earning more points | ||||
| Description | This was only tested on polls. But if I create a poll, then it later gets unvalidated, then I revalidate it... Composr acts like I added another poll and syndicates activity / awards more points. | ||||
| Steps To Reproduce | 1. Add a poll (make sure there's at least 2 that exist). 2. Delete the poll that is in use. 3. Revalidate / make in use one of the other existing polls. You are again awarded points / activity is syndicated as if the poll was new. | ||||
| Tags | Good for training | ||||
| Attach Tags | |||||
| Time estimation (hours) | 2 | ||||
| Sponsorship open | |||||
|
|
Yes, this is a limitation of how it works, there's no tracking to points being awarded and validation is assumed to happen only once. I'll keep this here as a possible improving, for us to add that tracking. |
|
|
Assigned to Patrick, as more integrity around points will be important with our AutonomousOrganization changes. I think it'd be pretty easy - just look to see if an Action Log message exists for validating that content type+ID combination, and if so, don't give points again. |
|
|
I had misunderstood this issue. I had thought that we were assigning points to validated content in Composr, in general. Actually point awarding has nothing to do with content validation. Polls is a special case though because you get points when the poll is SET, which is piggy-backed on the validation system. The issue in general raised a lot of questions, and Patrick and I discussed. I decided to just handle it myself. 1) This issue is specific to polls. It should not be a thing that non-privileged users are setting polls. The polls CMS is based on mid-impact permissions which generally makes sense (we want to encourage user submission of this content on many community sites), but not for SETTING the poll, or for EDITING IT. So I have resolved this issue by having the default installation override the mid-impact validation/self-content-editing privileges for poll CMS to be off for any non-staff usergroup. 2) We questioned why we assign points based on submission, not based on submitting-as-validated/subsequent-validation, and tracking it so that the points are only assigned once. The reasoning is that we are not trying to reward ACCEPTABLE submissions, we are trying to award EFFORT. If staff then want to come in and give extra points, or take some points away for spammy content (which they can easily do on the member punish form), they have the choice to do this. 3) We questioned why there is no action-log tracking for when content is validated. Tracking of all changes would be achieved via 2119, so we shouldn't be looking into adding specific validation logging at this point. |
|
|
Actually I am also going to make it not award points for a poll that was already set. It's not hard to do as a special case. I'm not going to skip re-syndication though, as an old poll can legitimately be put back with an expectation it will re-syndicate. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-08-28 06:43 | PDStig | New Issue | |
| 2016-08-28 15:28 | Chris Graham | Note Added: 0004283 | |
| 2016-09-05 18:40 | Chris Graham | Time estimation (hours) | => 2 |
| 2017-11-26 16:11 | Chris Graham | Tag Attached: Can implement during live chat | |
| 2019-06-27 19:42 | Chris Graham | Tag Renamed | Can implement during live chat => Good for training |
| 2022-08-03 01:47 | Chris Graham | Assigned To | => user4172 |
| 2022-08-03 01:47 | Chris Graham | Status | Not Assigned => Assigned |
| 2022-08-03 01:47 | Chris Graham | Tag Attached: Roadmap: v12 | |
| 2022-08-03 01:48 | Chris Graham | Note Added: 0007406 | |
| 2022-08-09 21:18 | Chris Graham | Status | Assigned => Resolved |
| 2022-08-09 21:18 | Chris Graham | Resolution | open => fixed |
| 2022-08-09 21:18 | Chris Graham | Note Added: 0007415 | |
| 2022-08-09 21:55 | Chris Graham | Note Added: 0007416 | |
| 2022-08-15 15:46 | Chris Graham | Tag Detached: Roadmap: v12 |
