Changeset 4299 for sandbox

Timestamp:

06/19/2008 01:01:16 PM (3 months ago)

Author:

zero

Message:

응답형 댓글의 잘못된 반업데이트 로직 수정

Location:

sandbox/modules/comment

Files:

• comment.controller.php (modified) (1 diff)
• queries/getCommentListItem.xml (modified) (1 diff)
• queries/getCommentPageList.xml (modified) (1 diff)
• queries/getCommentParentNextSibling.xml (added)
• queries/updateCommentListArrange.xml (modified) (1 diff)

sandbox/modules/comment/comment.controller.php

@@ -160 +160 @@
                 $list_args->head = $parent->head;
                 $list_args->depth = $parent->depth+1;
-                if($list_args->depth<2) $list_args->arrange = $obj->comment_srl;
-                else {
-                    $list_args->arrange = $parent->arrange;
-                    $output = executeQuery('comment.updateCommentListArrange', $list_args);
-                    if(!$output->toBool()) return $output;
+
+                // depth가 2단계 미만이면 별도의 update문 없이 insert만으로 쓰레드 정리
+                if($list_args->depth<2) {
+                    $list_args->arrange = $obj->comment_srl;
+
+                // depth가 2단계 이상이면 반업데이트 실행
+                } else {
+                    // 부모 댓글과 같은 head를 가지고 depth가 같거나 작은 댓글중 제일 위 댓글을 구함
+                    $p_args->head = $parent->head;
+                    $p_args->arrange = $parent->arrange;
+                    $p_args->depth = $parent->depth;
+                    $output = executeQuery('comment.getCommentParentNextSibling', $p_args);
+
+                    if($output->data->arrange) {
+                        $list_args->arrange = $output->data->arrange;
+                        $output = executeQuery('comment.updateCommentListArrange', $list_args);
+                    } else {
+                        $list_args->arrange = $obj->comment_srl;
+                    }
+
                 }
             }

sandbox/modules/comment/queries/getCommentListItem.xml

@@ -1 +1 @@
 <query id="getCommentListItem" action="select">
     <tables>
+        <table name="comments" />
         <table name="comments_list" />
     </tables>
     <columns>
-        <column name="*" />
+        <column name="comments.parent_srl" />
+        <column name="comments_list.*" />
     </columns>
     <conditions>
-        <condition operation="equal" column="comment_srl" var="comment_srl" notnull="notnull" />
+        <condition operation="equal" column="comments.comment_srl" var="comment_srl" notnull="notnull" />
+        <condition operation="equal" column="comments.comment_srl" default="comments_list.comment_srl" notnull="notnull" pipe="and" />
     </conditions>
 </query>

sandbox/modules/comment/queries/getCommentPageList.xml

@@ -6 +6 @@
     <columns>
         <column name="comments.*" />
-        <column name="depth" alias="depth" />
+        <column name="comments_list.depth" alias="depth" />
     </columns>
     <conditions>
         <condition operation="equal" column="comments_list.document_srl" var="document_srl" notnull="notnull" pipe="and" />
         <condition operation="equal" column="comments_list.comment_srl" var="comments.comment_srl" filter="number" pipe="and" />
+        <condition operation="more" column="comments_list.head" default="0" pipe="and" />
+        <condition operation="more" column="comments_list.arrange" default="0" pipe="and" />
     </conditions>
     <navigation>
-        <index var="sort_index" default="head" order="asc" />
-        <index var="sort_index" default="arrange" order="asc" />
-        <index var="sort_index" default="comment_srl" order="asc" />
         <list_count var="list_count" default="list_count" />
         <page_count var="page_count" default="10" />

sandbox/modules/comment/queries/updateCommentListArrange.xml

@@ -9 +9 @@
                 <condition operation="equal" column="document_srl" var="document_srl" filter="number" notnull="notnull" />
                 <condition operation="equal" column="head" var="head" filter="number" notnull="notnull" pipe="and" />
-                <condition operation="excess" column="arrange" var="arrange" filter="number" notnull="notnull" pipe="and" />
+                <condition operation="more" column="arrange" var="arrange" filter="number" notnull="notnull" pipe="and" />
         </conditions>
 </query>