svn commit: r1221347 - /ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1221347 - /ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl

ashish-18
Author: ashish
Date: Tue Dec 20 17:16:12 2011
New Revision: 1221347

URL: http://svn.apache.org/viewvc?rev=1221347&view=rev
Log:
Minor bug fixes on Pending Reviews page
1) Added <td> to show value of Review By  column in proper place.
2) Added default value 'N' for isAnonymous drop down option
3) Added additional condition to show valid value in Review By Column instead of showing value from previous record value.
Thanks Arun!

Modified:
    ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl

Modified: ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl?rev=1221347&r1=1221346&r2=1221347&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl (original)
+++ ofbiz/trunk/applications/product/webapp/catalog/reviews/pendingReviews.ftl Tue Dec 20 17:16:12 2011
@@ -74,19 +74,21 @@ under the License.
                           ${review.postedDateTime?if_exists}
                       </td>
                       <td>
-                      <#if postedPerson?has_content>
+                      <#if review.userLoginId?has_content && postedPerson?has_content>
+                        <td>
                         <#if postedPerson.firstName?has_content && postedPerson.lastName?has_content>
                             ${postedPerson.firstName} ${postedPerson.lastName}
                         <#else>
                             ${postedPerson.groupName}
                         </#if>
+                        </td>
                       <#else>
                           <td></td>
                       </#if>
                       </td>
                       <td>
                           <select name='postedAnonymous_o_${rowCount}'>
-                              <option>${review.postedAnonymous?default("N")}</option>
+                              <option value="${review.postedAnonymous?default("N")}">${review.postedAnonymous?default("N")}</option>
                               <option value="${review.postedAnonymous?default("N")}">----</option>
                               <option value="N">${uiLabelMap.CommonN}</option>
                               <option value="Y">${uiLabelMap.CommonY}</option>