Changeset 12767 for trunk

Timestamp:

Dec 19, 2011, 6:08:49 PM (12 years ago)

Author:

mistic100

Message:

feature:2538 transmits messages after comment edition trough SESSION (no more ugly url displayed, avoid some minor bugs)

Location:

trunk

Files:

• include/functions_session.inc.php (modified) (1 diff)
• include/picture_comment.inc.php (modified) (1 diff)
• picture.php (modified) (2 diffs)

trunk/include/functions_session.inc.php

@@ -142 +142 @@
 REPLACE INTO '.SESSIONS_TABLE.'
   (id,data,expiration)
-  VALUES(\''.get_remote_addr_session_hash().$session_id.'\',\''.$data.'\',now())
+  VALUES(\''.get_remote_addr_session_hash().$session_id.'\',\''.str_replace("'", "\'", $data).'\',now())
 ;';
   pwg_query($query);

trunk/include/picture_comment.inc.php

@@ -229 +229 @@
   {
     $key = get_ephemeral_key(3, $page['image_id']);
-    $content = '';
-    if ('reject'===@$comment_action)
-    {
-      $content = htmlspecialchars( stripslashes($comm['content']) );
-    }
     $template->assign('comment_add',
         array(
           'F_ACTION' => $url_self,
           'KEY' => $key,
-          'CONTENT' => $content,
+          'CONTENT' => null,
           'SHOW_AUTHOR' => !is_classic_user()
         ));

trunk/picture.php

@@ -321 +321 @@
       check_input_parameter('comment_to_edit', $_GET, false, PATTERN_ID);
       $author_id = get_comment_author_id($_GET['comment_to_edit']);
+      
       if (can_manage_comment('edit', $author_id))
       {
@@ -334 +335 @@
             );
 
+          $perform_redirect = false;
           switch ($comment_action)
           {
             case 'moderate':
-              array_push($page['infos'], l10n('An administrator must authorize your comment before it is visible.'));
+              $_SESSION['page_infos'][] = l10n('An administrator must authorize your comment before it is visible.');
             case 'validate':
-              array_push($page['infos'], l10n('Your comment has been registered'));
+              $_SESSION['page_infos'][] = l10n('Your comment has been registered');
+              $perform_redirect = true;
               break;
             case 'reject':
-              set_status_header(403);
-              array_push($page['errors'], l10n('Your comment has NOT been registered because it did not pass the validation rules'));
+              $_SESSION['page_errors'][] = l10n('Your comment has NOT been registered because it did not pass the validation rules');
+              $perform_redirect = true;
               break;
             default:
               trigger_error('Invalid comment action '.$comment_action, E_USER_WARNING);
           }
-            
+          
+          if ($perform_redirect)
+          {
+            redirect($url_self);
+          }
           unset($_POST['content']);
           break;