Ignore:
Timestamp:
2012/12/15 19:08:08 (9 years ago)
Author:
Seasoft
Message:

#1985 (LC_Page_Admin_Total#lfCheckError バリデーション対象を誤っている)
#1903 (無駄な処理を改善する)

  • ループ処理を使う必要が無いシーンで使って可読性が低くなっていた。
  • バリデーションで無駄なチェックを実行していた
  • <input name="form"> の渡し忘れがある。自前で複雑なことをして誤りを生んでいる。「search_」を前置して、共通処理で透過的に処理させる。
File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/version-2_12-dev/data/Smarty/templates/admin/total/index.tpl

    r22003 r22138  
    3232                    <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" /> 
    3333                    <input type="hidden" name="mode" value="search" /> 
    34                     <input type="hidden" name="form" value="1" /> 
     34                    <input type="hidden" name="search_form" value="1" /> 
    3535                    <input type="hidden" name="page" value="<!--{$arrForm.page.value|h}-->" /> 
    3636                    <input type="hidden" name="type" value="<!--{$smarty.post.type|h}-->" /> 
     
    5555                    <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" /> 
    5656                    <input type="hidden" name="mode" value="search" /> 
    57                     <input type="hidden" name="form" value="2" /> 
     57                    <input type="hidden" name="search_form" value="2" /> 
    5858                    <input type="hidden" name="page" value="<!--{$arrForm.page.value|h}-->" /> 
    5959                    <input type="hidden" name="type" value="<!--{$smarty.post.type|h}-->" /> 
     
    9999        <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" /> 
    100100        <input type="hidden" name="mode" value="search" /> 
    101         <input type="hidden" name="form" value="<!--{$arrForm.form.value|h}-->" /> 
    102101        <input type="hidden" name="type" value="<!--{$arrForm.type.value|h}-->" /> 
    103102        <input type="hidden" name="page" value="<!--{$arrForm.page.value|h}-->" /> 
     
    117116                <div id="graph-image"> 
    118117                    <!--{* <img src="<!--{$tpl_image}-->?<!--{$cashtime}-->" alt="グラフ"> *}--> 
    119                     <img src="?draw_image=true&amp;type=<!--{$smarty.post.type|h}-->&amp;mode=search&amp;page=<!--{$smarty.post.page|h}-->&amp;search_startyear_m=<!--{$smarty.post.search_startyear_m|h}-->&amp;search_startmonth_m=<!--{$smarty.post.search_startmonth_m|h}-->&amp;search_startyear=<!--{$smarty.post.search_startyear|h}-->&amp;search_startmonth=<!--{$smarty.post.search_startmonth|h}-->&amp;search_startday=<!--{$smarty.post.search_startday|h}-->&amp;search_endyear=<!--{$smarty.post.search_endyear|h}-->&amp;search_endmonth=<!--{$smarty.post.search_endmonth|h}-->&amp;search_endday=<!--{$smarty.post.search_endday|h}-->" alt="グラフ" /> 
     118                    <img src="?draw_image=true&amp;type=<!--{$smarty.post.type|h}-->&amp;mode=search&amp;search_form=<!--{$smarty.post.search_form|h}-->&amp;page=<!--{$smarty.post.page|h}-->&amp;search_startyear_m=<!--{$smarty.post.search_startyear_m|h}-->&amp;search_startmonth_m=<!--{$smarty.post.search_startmonth_m|h}-->&amp;search_startyear=<!--{$smarty.post.search_startyear|h}-->&amp;search_startmonth=<!--{$smarty.post.search_startmonth|h}-->&amp;search_startday=<!--{$smarty.post.search_startday|h}-->&amp;search_endyear=<!--{$smarty.post.search_endyear|h}-->&amp;search_endmonth=<!--{$smarty.post.search_endmonth|h}-->&amp;search_endday=<!--{$smarty.post.search_endday|h}-->" alt="グラフ" /> 
    120119                </div> 
    121120                <!--{/if}--> 
Note: See TracChangeset for help on using the changeset viewer.