Merge pull request 'Add link to the episode the comment is on' (#80) from I79_Add_link_to_the_comment_episode into main

Reviewed-on: HPR/hpr_hub#80

hub/ccdn.php

@@ -8,7 +8,7 @@ require "/home/hpr/php/include.php";
 
 $ip = $_SERVER["REMOTE_ADDR"];
 
-if ( $_SERVER['REQUEST_METHOD'] !== 'GET' ) {
+if ( $_SERVER['REQUEST_METHOD'] !== 'GET' && $_SERVER['REQUEST_METHOD'] !== 'HEAD' ) {
   naughty("34522f7b9ba88b319afa9f063b98905c");
 }
 

hub/comment_confirm.php

@@ -25,8 +25,6 @@ if (strcasecmp('public', $_POST["anti_spam_question"]) !== 0) {
     naughty("6aef421ce05e3ac34f4cd91ae3248a45");
 }
 
-$comment_directory = "/home/hpr/comments";
-
 if ( ! file_exists( $comment_directory ) ) {
     # Looks like the comments directory has not been created
     naughty("d5342ea497f701656433e81fb5eed064");
@@ -273,7 +271,7 @@ $mailer->addBCC('admin@hobbypublicradio.org');
 $mailer->AddAddress('comments@hackerpublicradio.org');
 $mailer->isHTML(false);
 $mailer->Subject = "New Comment for show hpr${eps_id} on ${ep_date} ${key}";
-$mailer->MsgHTML("<p>hpr${eps_id} on ${ep_date} by ${host} with the title <strong>${title}</strong> \"${summary}\"</p>
+$mailer->MsgHTML("<p><a href=\"https://hackerpublicradio.org/eps/hpr${eps_id}\">hpr${eps_id}</a> on ${ep_date} by ${host} with the title <strong>${title}</strong> \"${summary}\"</p>
 <p>
 See attachment for the json comment file.
 </p>
@@ -318,6 +316,8 @@ $justification
 );
 $mailer->AltBody = "hpr${eps_id} on ${ep_date} by ${host} with the title ${title} \"${summary}\"</p>
 
+https://hackerpublicradio.org/eps/hpr${eps_id}
+
 See attachment for the json comment file.
 
 There are now ${unprocessed_comments} unprocessed comments.

hub/upload_confirm.php

@@ -160,7 +160,7 @@ logextra( "license is a valid value" );
 // }
 // logextra( "Notes are less than max" );
 
-$notes_length = strlen($_POST["notes"])
+$notes_length = strlen($_POST["notes"]);
 logextra( "Notes are $notes_length long." );
 
 if ( ( empty($_POST["series"]) and ($_POST["series"] != 0 ) ) or (strlen($_POST["series"]) > 3 ) ) {
@@ -460,7 +460,7 @@ $mailer->addBCC('admin@hackerpublicradio.org');
 $mailer->addBCC('admin@hobbypublicradio.org');
 $mailer->AddAddress("$db_email");
 $mailer->isHTML(false);
-if ( $ep_num  === 9999 ) {
+if ( $ep_num  == "9999" ) {
   $mailer->Subject = "Thank you for uploading to the Reserve Queue";
   $mailer->MsgHTML("<p><em>This email is an automatic reply. If you have not made this request then please ignore this email.</em></p>
   <p>Thank You for recording an episode for the Reserve Queue.</p>
@@ -476,20 +476,20 @@ if ( $ep_num  === 9999 ) {
 else {
   $mailer->Subject = "Thank you for uploading hpr${ep_num}::${ep_date}";
   $mailer->MsgHTML("<p><em>This email is an automatic reply. If you have not made this request then please ignore this email.</em></p>
-  <p>Thank You for recording hpr${ep_num} for release on ${ep_date}.</p>
+  <p>Thank you for recording hpr${ep_num} for release on ${ep_date}.</p>
   <pre>
   $message
   </pre>
   <p>
   Your show will now be processed by a HPR Volunteer.<br />
   Thanks,<br />
-  HPR Bot
+  HPR Bot.
   </p>");
 }
 
 $mailer->AltBody = "This email is an automatic reply. If you have not made this request then please ignore this email.
 
-Thank You for recording hpr${ep_num} for release on ${ep_date}.
+Thank you for recording hpr${ep_num}, for release on ${ep_date}.
 
 $message