Comments outside the current window should have extra questions #124

New Issue

ken_fallon · 2023-06-17T18:20:08Z

ken_fallon commented

2023-06-17 18:20:08 +00:00

Current situation

Currently the comments system only has one challenge question when filling a comment.

The file content-episode.tpl.html has the following code for all episodes. This should only be for current episodes - namely those in the 10 day feed.

Requested change

Spammers use the tactic of adding spam to older shows. We are less strict on current shows, and for older ones we need to ensure that the additional three questions are exposed.

The submit form expects these to be filled out and will not accept the comment as the justification will not be filled.

For any future episodes ( > now() ) and episodes older than 10 shows ago ( episode.id - 10 ) have the following questions rendered.

<tr>
  <td>Are you a spammer → </td>
  <td>
  <input required checked="checked" type="radio" name="spammer" id="spammer_yes" value="Yes">
  <label for="spammer_yes">Yes</label>
  <input required type="radio" name="spammer" id="spammer_no" value="No">
  <label for="spammer_no">No</label>
  </td>
</tr>

<tr>
  <td>Who hosted this show → </td>
  <td>
  <select required name="hostid"><option value="" "selected">Select a Host</option>
  <!-- drop down list of hosts from the hpr_hpr.hosts table "SELECT hostid, host FROM hosts ORDER BY host ASC" -->
  </select>
<td>

<tr>
  <td>What does HPR mean to you ?</td>
  <td><textarea required name="justification" maxlength="200" rows="4" cols="50" placeholder="Convince us you are part of the community."></textarea></td>
</tr>

## Current situation Currently the comments system only has one challenge question when filling a comment. The file content-episode.tpl.html has the following code for all episodes. This should only be for **current episodes** - namely those in the 10 day feed. <input type="hidden" name="spammer" value="No"> <input type="hidden" name="hostid" value=""> <input type="hidden" name="justification" value="No justification is asked for or required."> <input type="hidden" name="eps_id" value=""> ## Requested change Spammers use the tactic of adding spam to older shows. We are less strict on current shows, and for older ones we need to ensure that the additional three questions are exposed. The submit form expects these to be filled out and will not accept the comment as the justification will not be filled. For any future episodes ( `> now()` ) and episodes older than 10 shows ago ( `episode.id - 10` ) have the following questions rendered. ``` <tr> <td>Are you a spammer → </td> <td> <input required checked="checked" type="radio" name="spammer" id="spammer_yes" value="Yes"> <label for="spammer_yes">Yes</label> <input required type="radio" name="spammer" id="spammer_no" value="No"> <label for="spammer_no">No</label> </td> </tr> <tr> <td>Who hosted this show → </td> <td> <select required name="hostid"><option value="" "selected">Select a Host</option>  </select> <td> <tr> <td>What does HPR mean to you ?</td> <td><textarea required name="justification" maxlength="200" rows="4" cols="50" placeholder="Convince us you are part of the community."></textarea></td> </tr> ```

rho_n was assigned by ken_fallon

2023-06-17 18:20:17 +00:00

rho_n added the

Bug Found

label 2023-06-18 11:14:19 +00:00

davmo self-assigned this 2023-07-23 15:36:04 +00:00

rho_n was unassigned by davmo

2023-07-23 15:36:04 +00:00

davmo referenced this issue

2023-07-23 15:52:39 +00:00

Various bug fixes #145

davmo added a new dependency 2023-07-23 15:55:08 +00:00