| 5357 |
dpurdie |
1 |
<%@LANGUAGE="VBSCRIPT"%>
|
|
|
2 |
<%
|
|
|
3 |
Option explicit
|
|
|
4 |
Response.Expires = 0 ' always load the page, dont store
|
|
|
5 |
%>
|
|
|
6 |
<%
|
|
|
7 |
'=====================================================
|
|
|
8 |
' Code Review
|
|
|
9 |
'=====================================================
|
|
|
10 |
%>
|
|
|
11 |
<!--#include file="common/conf.asp"-->
|
|
|
12 |
<!--#include file="common/globals.asp"-->
|
|
|
13 |
<!--#include file="common/formating.asp"-->
|
|
|
14 |
<!--#include file="common/qstr.asp"-->
|
|
|
15 |
<!--#include file="common/common_subs.asp"-->
|
|
|
16 |
<!--#include file="common/common_dbedit.asp"-->
|
|
|
17 |
<!--#include file="common/_popup_window_common.asp"-->
|
|
|
18 |
<%
|
|
|
19 |
' Set rfile parameter. This is a return page after Login
|
|
|
20 |
Call objPMod.StoreParameter ( "rfile", "fixed_issues.asp" )
|
|
|
21 |
'------------ ACCESS CONTROL ------------------
|
|
|
22 |
%>
|
|
|
23 |
<!--#include file="_access_control_login.asp"-->
|
|
|
24 |
<!--#include file="_access_control_general.asp"-->
|
|
|
25 |
<!--#include file="_access_control_project.asp"-->
|
|
|
26 |
<%
|
|
|
27 |
'------------ Variable Definition -------------
|
|
|
28 |
Dim parPv_id
|
|
|
29 |
Dim rsQry
|
|
|
30 |
Dim pkgName
|
|
|
31 |
Dim pkgVersion
|
|
|
32 |
Dim isDLocked
|
|
|
33 |
Dim pageIsEditable
|
|
|
34 |
'------------ Constants Declaration -----------
|
|
|
35 |
'------------ Variable Init -------------------
|
|
|
36 |
parPv_id = Request("pv_id")
|
|
|
37 |
parRtag_id = Request("rtag_id")
|
|
|
38 |
'----------------------------------------------
|
|
|
39 |
%>
|
|
|
40 |
<%
|
|
|
41 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
42 |
Sub GetCodeReview ( nPv_id, oRS )
|
|
|
43 |
Dim Query_String
|
|
|
44 |
Query_String = _
|
|
|
45 |
" SELECT * FROM CODE_REVIEWS WHERE pv_id = :PV_ID"
|
|
|
46 |
Query_String = Replace( Query_String, ":PV_ID", nPv_id )
|
|
|
47 |
|
|
|
48 |
Set oRS = OraDatabase.CreateDynaset( Query_String, cint(0))
|
|
|
49 |
End Sub
|
|
|
50 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
51 |
Function DecodeItemResult ( cCheck )
|
|
|
52 |
|
|
|
53 |
Select Case cCheck
|
|
|
54 |
Case enumCODE_REVIEW_PASS
|
|
|
55 |
DecodeItemResult = "<option>Pass</option>"
|
|
|
56 |
Case enumCODE_REVIEW_FAIL
|
|
|
57 |
DecodeItemResult = "<option>Fail</option>"
|
|
|
58 |
Case Else
|
|
|
59 |
DecodeItemResult = "<option></option>"
|
|
|
60 |
End Select
|
|
|
61 |
|
|
|
62 |
End Function
|
|
|
63 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
64 |
Function DecodeOverallResult ( ByVal cCheck )
|
|
|
65 |
If cCheck = "" OR IsNull(cCheck) Then cCheck = 0
|
|
|
66 |
|
|
|
67 |
Select Case CInt( cCheck )
|
|
|
68 |
Case enumCODE_REVIEW_ACCEPTED
|
|
|
69 |
DecodeOverallResult = "<option>Accepted</option>"
|
|
|
70 |
Case enumCODE_REVIEW_MINOR_UPDATES
|
|
|
71 |
DecodeOverallResult = "<option>Minor updates required</option>"
|
|
|
72 |
Case enumCODE_REVIEW_MAJOR_REWORK
|
|
|
73 |
DecodeOverallResult = "<option>Major rework required</option>"
|
|
|
74 |
Case Else
|
|
|
75 |
DecodeOverallResult = "<option></option>"
|
|
|
76 |
End Select
|
|
|
77 |
|
|
|
78 |
End Function
|
|
|
79 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
80 |
Function ConstructSQL ( nPv_id, sQuery )
|
|
|
81 |
Dim SQLqry, arrLineSplit, arrParamSplit, line
|
|
|
82 |
Const localPARAM_SPLIT = ":"
|
|
|
83 |
|
|
|
84 |
SQLqry = ReadFile( sQuery )
|
|
|
85 |
|
|
|
86 |
arrLineSplit = Split ( SQLqry, VBNewLine )
|
|
|
87 |
|
|
|
88 |
For Each line in arrLineSplit
|
|
|
89 |
arrParamSplit = Split ( line, localPARAM_SPLIT ) ' Split the line with ":" e.g. some_column = ':param_name:',
|
|
|
90 |
If Ubound( arrParamSplit ) > 0 Then
|
|
|
91 |
SQLqry = Replace( SQLqry, localPARAM_SPLIT & arrParamSplit( 1 ) & localPARAM_SPLIT, Replace( Request( arrParamSplit( 1 ) ), "'", "''") )
|
|
|
92 |
End If
|
|
|
93 |
Next
|
|
|
94 |
|
|
|
95 |
ConstructSQL = SQLqry
|
|
|
96 |
|
|
|
97 |
End Function
|
|
|
98 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
99 |
Sub UpdateCodeReview ( nPv_id )
|
|
|
100 |
Dim SQLstr
|
|
|
101 |
SQLstr = ConstructSQL ( nPv_id, rootPath & "queries\update_code_review.sql" )
|
|
|
102 |
|
|
|
103 |
objEH.TryORA ( OraSession )
|
|
|
104 |
On Error Resume Next
|
|
|
105 |
|
|
|
106 |
OraDatabase.ExecuteSQL SQLstr
|
|
|
107 |
|
|
|
108 |
objEH.CatchORA ( OraSession )
|
|
|
109 |
End Sub
|
|
|
110 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
111 |
%>
|
|
|
112 |
<%
|
|
|
113 |
'------------ Action Requirements -------------------
|
|
|
114 |
'Process submition
|
|
|
115 |
If CBool(Request("action")) AND objAccessControl.UserLogedIn Then
|
|
|
116 |
|
|
|
117 |
Call UpdateCodeReview ( parPv_id )
|
|
|
118 |
Call OpenInParentWindow ("fixed_issues.asp?pv_id="& parPv_id &"&rtag_id="& parRtag_id )
|
|
|
119 |
'Call CloseWindow
|
|
|
120 |
|
|
|
121 |
End If
|
|
|
122 |
|
|
|
123 |
Call Get_Pkg_Short_Info( parPv_id, NULL, pkgName, pkgVersion, NULL, NULL, isDLocked )
|
|
|
124 |
Call GetCodeReview ( parPv_id, rsQry )
|
|
|
125 |
pageIsEditable = Is_Page_Editable ( isDLocked )
|
|
|
126 |
%>
|
|
|
127 |
<html>
|
|
|
128 |
<head>
|
|
|
129 |
<title>Release Manager</title>
|
|
|
130 |
<link rel="shortcut icon" href="<%=FavIcon%>"/>
|
|
|
131 |
<meta HTTP-EQUIV="Pragma" CONTENT="no-cache">
|
|
|
132 |
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
|
|
|
133 |
<link rel="stylesheet" href="images/release_manager_style.css" type="text/css">
|
|
|
134 |
<link rel="stylesheet" href="images/navigation.css" type="text/css">
|
|
|
135 |
<script language="JavaScript" src="images/common.js"></script>
|
|
|
136 |
<!-- TIPS -->
|
|
|
137 |
<script language="JavaScript" src="images/tipster.js"></script>
|
|
|
138 |
<script language="JavaScript" src="images/_help_tips.js"></script>
|
|
|
139 |
</head>
|
|
|
140 |
<body bgcolor="#FFFFFF" text="#000000" leftmargin="0" topmargin="0" onload="self.focus();">
|
|
|
141 |
<!-- TIPS LAYERS -------------------------------------->
|
|
|
142 |
<div id="formTipsLayer" style="position: absolute; z-index: 1000; visibility: hidden; left:0; top: 0; width: 10"> </div>
|
|
|
143 |
<!----------------------------------------------------->
|
|
|
144 |
<form action="_wform_update_code_review.asp" method="post" name="codereview">
|
|
|
145 |
<table width="100%" border="0" cellspacing="0" cellpadding="2" height="100%">
|
|
|
146 |
<tr>
|
|
|
147 |
<td width="1%" background="images/lbox_bg_orange.gif"><img src="images/s_code_review_off.gif" width="21" height="23" hspace="5" border="0"></td>
|
|
|
148 |
<td width="1%" background="images/lbox_bg_blue.gif" nowrap class="wform_ttl"> Code Review for <%=pkgName%> <%=pkgVersion%> </td>
|
|
|
149 |
<td width="100%" background="images/lbox_bg_blue.gif" align="right" nowrap class="wform_ttl">
|
|
|
150 |
<input type="submit" name="btn" value="Edit" <%If pageIsEditable Then%>class="form_btn"<%Else%>disabled class="form_btn_disabled"<%End If%>>
|
|
|
151 |
<input type="reset" name="btn" value="Close" class="form_btn" onclick="self.close()">
|
|
|
152 |
</td>
|
|
|
153 |
<td background="images/lbox_bg_blue.gif" align="right" width="1%" nowrap><img src="images/h_trsp_dot.gif" width="5" height="22"></td>
|
|
|
154 |
</tr>
|
|
|
155 |
<tr>
|
|
|
156 |
<td height="100%" width="1%"> </td>
|
|
|
157 |
<td valign="top" nowrap colspan="3" class="wform_ttl" background="images/bg_form_lightgray.gif">
|
|
|
158 |
<br>
|
|
|
159 |
<br>
|
|
|
160 |
<table width="100%" border="0" cellspacing="0" cellpadding="0">
|
|
|
161 |
<tr>
|
|
|
162 |
<td width="1%"><img src="images/h_trsp_dot.gif" width="10" height="30"></td>
|
|
|
163 |
<td width="100%">
|
|
|
164 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
165 |
<tr align="center">
|
|
|
166 |
<td colspan="4" class="form_ttl" background="images/bg_form_lightbluedark.gif">Summary</td>
|
|
|
167 |
</tr>
|
|
|
168 |
<tr>
|
|
|
169 |
<td width="1%" align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Date of Review</td>
|
|
|
170 |
<td width="1%" nowrap bgcolor="#FFFFFF" class="form_item"><%=EuroDate ( rsQry("date_of_review") )%> </td>
|
|
|
171 |
<td width="1%" align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field"> Time Spent</td>
|
|
|
172 |
<td width="100%" bgcolor="#FFFFFF" class="form_item"><%=rsQry("time_spent")%> hrs </td>
|
|
|
173 |
</tr>
|
|
|
174 |
<tr>
|
|
|
175 |
<td align="right" valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Reason for Review</td>
|
|
|
176 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("review_reason") )))%></td>
|
|
|
177 |
</tr>
|
|
|
178 |
<tr>
|
|
|
179 |
<td rowspan="4" align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Review Team </td>
|
|
|
180 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Domain Expert </td>
|
|
|
181 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rteam_domain_expert") )))%></td>
|
|
|
182 |
</tr>
|
|
|
183 |
<tr>
|
|
|
184 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Language Expert </td>
|
|
|
185 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rteam_language_expert") )))%></td>
|
|
|
186 |
</tr>
|
|
|
187 |
<tr>
|
|
|
188 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Peer Developer </td>
|
|
|
189 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rteam_peer_developer") )))%></td>
|
|
|
190 |
</tr>
|
|
|
191 |
<tr>
|
|
|
192 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Author</td>
|
|
|
193 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rteam_author") )))%></td>
|
|
|
194 |
</tr>
|
|
|
195 |
<tr>
|
|
|
196 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Files Reviewed</td>
|
|
|
197 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("files_reviewed") )))%></td>
|
|
|
198 |
</tr>
|
|
|
199 |
<tr>
|
|
|
200 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Review Results </td>
|
|
|
201 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item"><select name="review_results" disabled class="form_item" id="review_results">
|
|
|
202 |
<%=DecodeOverallResult( rsQry("review_results") )%>
|
|
|
203 |
</select></td>
|
|
|
204 |
</tr>
|
|
|
205 |
<tr>
|
|
|
206 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Issues Raised </td>
|
|
|
207 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("issues_raised") )))%></td>
|
|
|
208 |
</tr>
|
|
|
209 |
<tr>
|
|
|
210 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Comments</td>
|
|
|
211 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("review_comments") )))%></td>
|
|
|
212 |
</tr>
|
|
|
213 |
</table>
|
|
|
214 |
<br>
|
|
|
215 |
<br>
|
|
|
216 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
217 |
<tr align="center">
|
|
|
218 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Functionality</td>
|
|
|
219 |
</tr>
|
|
|
220 |
<tr>
|
|
|
221 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
222 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
223 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
224 |
</tr>
|
|
|
225 |
<tr>
|
|
|
226 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Meets required functionality </td>
|
|
|
227 |
<td valign="top" bgcolor="#FFFFFF"><select name="fnc_s_meets_functionality" disabled class="form_item" id="fnc_s_meets_functionality">
|
|
|
228 |
<%=DecodeItemResult( rsQry("fnc_s_meets_functionality") )%>
|
|
|
229 |
</select></td>
|
|
|
230 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("fnc_c_meets_functionality") )))%></td>
|
|
|
231 |
</tr>
|
|
|
232 |
</table>
|
|
|
233 |
<br>
|
|
|
234 |
<br>
|
|
|
235 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
236 |
<tr align="center">
|
|
|
237 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Robustness</td>
|
|
|
238 |
</tr>
|
|
|
239 |
<tr>
|
|
|
240 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
241 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
242 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
243 |
</tr>
|
|
|
244 |
<tr>
|
|
|
245 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Boundary conditions handled </td>
|
|
|
246 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_bound_cond_handled" disabled class="form_item" id="rbs_s_bound_cond_handled">
|
|
|
247 |
<%=DecodeItemResult( rsQry("rbs_s_bound_cond_handled") )%>
|
|
|
248 |
</select></td>
|
|
|
249 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_bound_cond_handled") )))%></td>
|
|
|
250 |
</tr>
|
|
|
251 |
<tr>
|
|
|
252 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Class interface preconditions tested</td>
|
|
|
253 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_class_interf_precond" disabled class="form_item" id="rbs_s_class_interf_precond">
|
|
|
254 |
<%=DecodeItemResult( rsQry("rbs_s_class_interf_precond") )%>
|
|
|
255 |
</select></td>
|
|
|
256 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_class_interf_precond") )))%></td>
|
|
|
257 |
</tr>
|
|
|
258 |
<tr>
|
|
|
259 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">No unititalised data</td>
|
|
|
260 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_no_unitialised_date" disabled class="form_item" id="rbs_s_no_unitialised_date">
|
|
|
261 |
<%=DecodeItemResult( rsQry("rbs_s_no_unitialised_date") )%>
|
|
|
262 |
</select></td>
|
|
|
263 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_no_unitialised_date") )))%></td>
|
|
|
264 |
</tr>
|
|
|
265 |
<tr>
|
|
|
266 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Exception handling</td>
|
|
|
267 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_excep_handing" disabled class="form_item" id="rbs_s_excep_handing">
|
|
|
268 |
<%=DecodeItemResult( rsQry("rbs_s_excep_handing") )%>
|
|
|
269 |
</select></td>
|
|
|
270 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_excep_handing") )))%></td>
|
|
|
271 |
</tr>
|
|
|
272 |
<tr>
|
|
|
273 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Resource mgt (db cxn, memory)</td>
|
|
|
274 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_resource_mng" disabled class="form_item" id="rbs_s_resource_mng">
|
|
|
275 |
<%=DecodeItemResult( rsQry("rbs_s_resource_mng") )%>
|
|
|
276 |
</select></td>
|
|
|
277 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_resource_mng") )))%></td>
|
|
|
278 |
</tr>
|
|
|
279 |
<tr>
|
|
|
280 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Transaction completeness</td>
|
|
|
281 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_trans_complet" disabled class="form_item" id="rbs_s_trans_complet">
|
|
|
282 |
<%=DecodeItemResult( rsQry("rbs_s_trans_complet") )%>
|
|
|
283 |
</select></td>
|
|
|
284 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_trans_complet") )))%></td>
|
|
|
285 |
</tr>
|
|
|
286 |
<tr>
|
|
|
287 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Threadsafety</td>
|
|
|
288 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_threadsafety" disabled class="form_item" id="rbs_s_threadsafety">
|
|
|
289 |
<%=DecodeItemResult( rsQry("rbs_s_threadsafety") )%>
|
|
|
290 |
</select></td>
|
|
|
291 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_threadsafety") )))%></td>
|
|
|
292 |
</tr>
|
|
|
293 |
<tr>
|
|
|
294 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Return values (where appropriate)</td>
|
|
|
295 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_ret_vals" disabled class="form_item" id="rbs_s_ret_vals">
|
|
|
296 |
<%=DecodeItemResult( rsQry("rbs_s_ret_vals") )%>
|
|
|
297 |
</select></td>
|
|
|
298 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_ret_vals") )))%></td>
|
|
|
299 |
</tr>
|
|
|
300 |
<tr>
|
|
|
301 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Correct err handling at Subsys bdys</td>
|
|
|
302 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_corr_err_handling" disabled class="form_item" id="rbs_s_corr_err_handling">
|
|
|
303 |
<%=DecodeItemResult( rsQry("rbs_s_corr_err_handling") )%>
|
|
|
304 |
</select></td>
|
|
|
305 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_corr_err_handling") )))%></td>
|
|
|
306 |
</tr>
|
|
|
307 |
<tr>
|
|
|
308 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">SQL correct and conforming to standards</td>
|
|
|
309 |
<td valign="top" bgcolor="#FFFFFF"><select name="rbs_s_sql_std" disabled class="form_item" id="rbs_s_sql_std">
|
|
|
310 |
<%=DecodeItemResult( rsQry("rbs_s_sql_std") )%>
|
|
|
311 |
</select></td>
|
|
|
312 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("rbs_c_sql_std") )))%></td>
|
|
|
313 |
</tr>
|
|
|
314 |
</table>
|
|
|
315 |
<br>
|
|
|
316 |
<br>
|
|
|
317 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
318 |
<tr align="center">
|
|
|
319 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Maintainability</td>
|
|
|
320 |
</tr>
|
|
|
321 |
<tr>
|
|
|
322 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
323 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
324 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
325 |
</tr>
|
|
|
326 |
<tr>
|
|
|
327 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">External references explicit (eg: pkg imports)</td>
|
|
|
328 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_ext_ref" disabled class="form_item" id="mnt_s_ext_ref">
|
|
|
329 |
<%=DecodeItemResult( rsQry("mnt_s_ext_ref") )%>
|
|
|
330 |
</select></td>
|
|
|
331 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_ext_ref") )))%></td>
|
|
|
332 |
</tr>
|
|
|
333 |
<tr>
|
|
|
334 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Class size ok</td>
|
|
|
335 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_class_size" disabled class="form_item" id="mnt_s_class_size">
|
|
|
336 |
<%=DecodeItemResult( rsQry("mnt_s_class_size") )%>
|
|
|
337 |
</select></td>
|
|
|
338 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_class_size") )))%></td>
|
|
|
339 |
</tr>
|
|
|
340 |
<tr>
|
|
|
341 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Method size ok</td>
|
|
|
342 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_method_size" disabled class="form_item" id="mnt_s_method_size">
|
|
|
343 |
<%=DecodeItemResult( rsQry("mnt_s_method_size") )%>
|
|
|
344 |
</select></td>
|
|
|
345 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_method_size") )))%></td>
|
|
|
346 |
</tr>
|
|
|
347 |
<tr>
|
|
|
348 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Appropriate commenting</td>
|
|
|
349 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_approp_comm" disabled class="form_item" id="mnt_s_approp_comm">
|
|
|
350 |
<%=DecodeItemResult( rsQry("mnt_s_approp_comm") )%>
|
|
|
351 |
</select></td>
|
|
|
352 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_approp_comm") )))%></td>
|
|
|
353 |
</tr>
|
|
|
354 |
<tr>
|
|
|
355 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Indentifiers conform to naming standards</td>
|
|
|
356 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_ident_name_std" disabled class="form_item" id="mnt_s_ident_name_std">
|
|
|
357 |
<%=DecodeItemResult( rsQry("mnt_s_ident_name_std") )%>
|
|
|
358 |
</select></td>
|
|
|
359 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_ident_name_std") )))%></td>
|
|
|
360 |
</tr>
|
|
|
361 |
<tr>
|
|
|
362 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Switch statements have defaults</td>
|
|
|
363 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_switch_have_defaults" disabled class="form_item" id="mnt_s_switch_have_defaults">
|
|
|
364 |
<%=DecodeItemResult( rsQry("mnt_s_switch_have_defaults") )%>
|
|
|
365 |
</select></td>
|
|
|
366 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_switch_have_defaults") )))%></td>
|
|
|
367 |
</tr>
|
|
|
368 |
<tr>
|
|
|
369 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">No literal numeric constants except 1, 0, 1</td>
|
|
|
370 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_no_lit_num" disabled class="form_item" id="mnt_s_no_lit_num">
|
|
|
371 |
<%=DecodeItemResult( rsQry("mnt_s_no_lit_num") )%>
|
|
|
372 |
</select></td>
|
|
|
373 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_no_lit_num") )))%></td>
|
|
|
374 |
</tr>
|
|
|
375 |
<tr>
|
|
|
376 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">No dead or unused code</td>
|
|
|
377 |
<td valign="top" bgcolor="#FFFFFF"><select name="mnt_s_no_dead_code" disabled class="form_item" id="mnt_s_no_dead_code">
|
|
|
378 |
<%=DecodeItemResult( rsQry("mnt_s_no_dead_code") )%>
|
|
|
379 |
</select></td>
|
|
|
380 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("mnt_c_no_dead_code") )))%></td>
|
|
|
381 |
</tr>
|
|
|
382 |
</table>
|
|
|
383 |
<br>
|
|
|
384 |
<br>
|
|
|
385 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
386 |
<tr align="center">
|
|
|
387 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Design Issues </td>
|
|
|
388 |
</tr>
|
|
|
389 |
<tr>
|
|
|
390 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
391 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
392 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
393 |
</tr>
|
|
|
394 |
<tr>
|
|
|
395 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Use of standard design patterns</td>
|
|
|
396 |
<td valign="top" bgcolor="#FFFFFF"><select name="dei_s_std_des_patt" disabled class="form_item" id="dei_s_std_des_patt">
|
|
|
397 |
<%=DecodeItemResult( rsQry("dei_s_std_des_patt") )%>
|
|
|
398 |
</select></td>
|
|
|
399 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("dei_c_std_des_patt") )))%></td>
|
|
|
400 |
</tr>
|
|
|
401 |
<tr>
|
|
|
402 |
<td valign="top" background="images/bg_form_lightbluedark.gif" class="form_item">Use of appropriate algorithms (consider performance)</td>
|
|
|
403 |
<td valign="top" bgcolor="#FFFFFF"><select name="dei_s_approp_algor" disabled class="form_item" id="dei_s_approp_algor">
|
|
|
404 |
<%=DecodeItemResult( rsQry("dei_s_approp_algor") )%>
|
|
|
405 |
</select></td>
|
|
|
406 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("dei_c_approp_algor") )))%></td>
|
|
|
407 |
</tr>
|
|
|
408 |
<tr>
|
|
|
409 |
<td valign="top" background="images/bg_form_lightbluedark.gif" class="form_item">Use of appropriate objects (consider bloat, object focus, encapsulation)</td>
|
|
|
410 |
<td valign="top" bgcolor="#FFFFFF"><select name="dei_s_approp_obj" disabled class="form_item" id="dei_s_approp_obj">
|
|
|
411 |
<%=DecodeItemResult( rsQry("dei_s_approp_obj") )%>
|
|
|
412 |
</select></td>
|
|
|
413 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("dei_c_approp_obj") )))%></td>
|
|
|
414 |
</tr>
|
|
|
415 |
<tr>
|
|
|
416 |
<td valign="top" background="images/bg_form_lightbluedark.gif" class="form_item">Appropriate user error messages</td>
|
|
|
417 |
<td valign="top" bgcolor="#FFFFFF"><select name="dei_s_approp_err_msg" disabled class="form_item" id="dei_s_approp_err_msg">
|
|
|
418 |
<%=DecodeItemResult( rsQry("dei_s_approp_err_msg") )%>
|
|
|
419 |
</select></td>
|
|
|
420 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("dei_c_approp_err_msg") )))%></td>
|
|
|
421 |
</tr>
|
|
|
422 |
<tr>
|
|
|
423 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Use of standard framework components</td>
|
|
|
424 |
<td valign="top" bgcolor="#FFFFFF"><select name="dei_s_std_framew_comp" disabled class="form_item" id="dei_s_std_framew_comp">
|
|
|
425 |
<%=DecodeItemResult( rsQry("dei_s_std_framew_comp") )%>
|
|
|
426 |
</select></td>
|
|
|
427 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("dei_c_std_framew_comp") )))%></td>
|
|
|
428 |
</tr>
|
|
|
429 |
<tr>
|
|
|
430 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Appropriate logging</td>
|
|
|
431 |
<td valign="top" bgcolor="#FFFFFF"><select name="dei_s_approp_logging" disabled class="form_item" id="dei_s_approp_logging">
|
|
|
432 |
<%=DecodeItemResult( rsQry("dei_s_approp_logging") )%>
|
|
|
433 |
</select></td>
|
|
|
434 |
<td bgcolor="#FFFFFF" class="form_item"><%=NewLine_To_BR( To_HTML( SQLstring (rsQry("dei_c_approp_logging") )))%></td>
|
|
|
435 |
</tr>
|
|
|
436 |
</table>
|
|
|
437 |
<br>
|
|
|
438 |
<br></td>
|
|
|
439 |
</tr>
|
|
|
440 |
</table></td>
|
|
|
441 |
</tr>
|
|
|
442 |
<tr>
|
|
|
443 |
<td height="1%" width="1%"><img src="images/h_trsp_dot.gif" width="5" height="5"></td>
|
|
|
444 |
<td valign="top" nowrap colspan="3" class="wform_ttl" background="images/lbox_bg_blue.gif" align="right">
|
|
|
445 |
<input type="submit" name="btn" value="Edit" <%If pageIsEditable Then%>class="form_btn"<%Else%>disabled class="form_btn_disabled"<%End If%>>
|
|
|
446 |
<input type="reset" name="btn" value="Close" class="form_btn" onclick="self.close()">
|
|
|
447 |
</td>
|
|
|
448 |
</tr>
|
|
|
449 |
</table>
|
|
|
450 |
<input type="hidden" name="pv_id" value="<%=parPv_id%>">
|
|
|
451 |
<input type="hidden" name="rtag_id" value="<%=parRtag_id%>">
|
|
|
452 |
</form>
|
|
|
453 |
</body>
|
|
|
454 |
</html>
|
|
|
455 |
|
|
|
456 |
|
|
|
457 |
<!-- DESTRUCTOR ------->
|
|
|
458 |
<!--#include file="common/destructor.asp"-->
|