| 5357 |
dpurdie |
1 |
<%@LANGUAGE="VBSCRIPT"%>
|
|
|
2 |
<%
|
|
|
3 |
Option explicit
|
|
|
4 |
Response.Expires = 0 ' always load the page, dont store
|
|
|
5 |
%>
|
|
|
6 |
<%
|
|
|
7 |
'=====================================================
|
|
|
8 |
' Update 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 arrItemResults
|
|
|
31 |
Dim arrOverallResults
|
|
|
32 |
Dim pkgName
|
|
|
33 |
Dim pkgVersion
|
|
|
34 |
'------------ Constants Declaration -----------
|
|
|
35 |
'------------ Variable Init -------------------
|
|
|
36 |
parPv_id = Request("pv_id")
|
|
|
37 |
parRtag_id = Request("rtag_id")
|
|
|
38 |
arrItemResults = Array ( "","","", _
|
|
|
39 |
"Pass",enumCODE_REVIEW_PASS,"class='pass_alert'", _
|
|
|
40 |
"Fail",enumCODE_REVIEW_FAIL,"class='err_alert'" )
|
|
|
41 |
arrOverallResults = Array ( "","", _
|
|
|
42 |
"Accepted",enumCODE_REVIEW_ACCEPTED, _
|
|
|
43 |
"Minor updates required",enumCODE_REVIEW_MINOR_UPDATES, _
|
|
|
44 |
"Major rework required",enumCODE_REVIEW_MAJOR_REWORK )
|
|
|
45 |
'----------------------------------------------
|
|
|
46 |
%>
|
|
|
47 |
<%
|
|
|
48 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
49 |
Sub GetCodeReview ( nPv_id, oRS )
|
|
|
50 |
Dim Query_String
|
|
|
51 |
Query_String = _
|
|
|
52 |
" SELECT * FROM CODE_REVIEWS WHERE pv_id = :PV_ID"
|
|
|
53 |
Query_String = Replace( Query_String, ":PV_ID", nPv_id )
|
|
|
54 |
|
|
|
55 |
Set oRS = OraDatabase.CreateDynaset( Query_String, cint(0))
|
|
|
56 |
|
|
|
57 |
' Make sure therer is always a record to read from
|
|
|
58 |
If oRS.RecordCount < 1 Then
|
|
|
59 |
oRS.Close
|
|
|
60 |
Call InserDefaultRow ( parPv_id )
|
|
|
61 |
Set oRS = OraDatabase.CreateDynaset( Query_String, cint(0))
|
|
|
62 |
End If
|
|
|
63 |
|
|
|
64 |
End Sub
|
|
|
65 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
66 |
Function DrawItemResult ( cCheck )
|
|
|
67 |
Dim item, val, maxOptions, selected, tempSTR
|
|
|
68 |
maxOptions = UBound( arrItemResults )
|
|
|
69 |
|
|
|
70 |
tempSTR = ""
|
|
|
71 |
For val = 0 to maxOptions Step 3
|
|
|
72 |
selected = ""
|
|
|
73 |
If CStr( arrItemResults(val + 1) ) = cCheck Then selected = "selected"
|
|
|
74 |
tempSTR = tempSTR & "<option value='"& arrItemResults( val + 1 ) &"' "& selected &" "& arrItemResults(val + 2) &">"& arrItemResults( val ) &"</option>"& VBNewLine
|
|
|
75 |
Next
|
|
|
76 |
|
|
|
77 |
DrawItemResult = tempSTR
|
|
|
78 |
End Function
|
|
|
79 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
80 |
Function DrawOverallResult ( ByVal nResult )
|
|
|
81 |
Dim val, maxOptions, tempSTR, selected
|
|
|
82 |
maxOptions = Ubound( arrOverallResults )
|
|
|
83 |
If nResult = "" OR IsNull(nResult) Then nResult = 0
|
|
|
84 |
|
|
|
85 |
tempSTR = ""
|
|
|
86 |
For val = 0 To maxOptions Step 2
|
|
|
87 |
selected = ""
|
|
|
88 |
If arrOverallResults(val + 1) = CInt( nResult ) Then selected = "selected"
|
|
|
89 |
tempSTR = tempSTR & "<option value='"& arrOverallResults( val + 1 ) &"' "& selected &">"& arrOverallResults( val ) &"</option>"& VBNewLine
|
|
|
90 |
Next
|
|
|
91 |
|
|
|
92 |
DrawOverallResult = tempSTR
|
|
|
93 |
End Function
|
|
|
94 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
95 |
Sub InserDefaultRow ( nPv_id )
|
|
|
96 |
objEH.TryORA ( OraSession )
|
|
|
97 |
On Error Resume Next
|
|
|
98 |
OraDatabase.ExecuteSQL _
|
|
|
99 |
" INSERT INTO CODE_REVIEWS (pv_id) VALUES("& nPv_id &")"
|
|
|
100 |
objEH.CatchORA ( OraSession )
|
|
|
101 |
End Sub
|
|
|
102 |
'-----------------------------------------------------------------------------------------------------------------------------------
|
|
|
103 |
%>
|
|
|
104 |
<%
|
|
|
105 |
'------------ Action Requirements -------------------
|
|
|
106 |
'Process submition
|
|
|
107 |
If CBool(Request("action")) AND objAccessControl.UserLogedIn Then
|
|
|
108 |
|
|
|
109 |
Call UpdateCodeReview ( parPv_id )
|
|
|
110 |
Call OpenInParentWindow ("fixed_issues.asp?pv_id="& parPv_id &"&rtag_id="& parRtag_id )
|
| 5957 |
dpurdie |
111 |
'Call Destroy_All_Objects
|
| 5357 |
dpurdie |
112 |
'Response.Redirect ( "_wform_code_review.asp?pv_id="& parPv_id &"&rtag_id="& parRtag_id )
|
|
|
113 |
'Call CloseWindow
|
|
|
114 |
|
|
|
115 |
End If
|
|
|
116 |
Call Get_Pkg_Short_Info( parPv_id, NULL, pkgName, pkgVersion, NULL, NULL, NULL )
|
|
|
117 |
Call GetCodeReview ( parPv_id, rsQry )
|
|
|
118 |
%>
|
|
|
119 |
<html>
|
|
|
120 |
<head>
|
|
|
121 |
<title>Release Manager</title>
|
|
|
122 |
<link rel="shortcut icon" href="<%=FavIcon%>"/>
|
|
|
123 |
<meta HTTP-EQUIV="Pragma" CONTENT="no-cache">
|
|
|
124 |
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
|
|
|
125 |
<link rel="stylesheet" href="images/release_manager_style.css" type="text/css">
|
|
|
126 |
<link rel="stylesheet" href="images/navigation.css" type="text/css">
|
|
|
127 |
<script language="JavaScript" src="images/common.js"></script>
|
|
|
128 |
<!-- TIPS -->
|
|
|
129 |
<script language="JavaScript" src="images/tipster.js"></script>
|
|
|
130 |
<script language="JavaScript" src="images/_help_tips.js"></script>
|
|
|
131 |
</head>
|
|
|
132 |
<body bgcolor="#FFFFFF" text="#000000" leftmargin="0" topmargin="0" onload="self.focus();">
|
|
|
133 |
<form action="_wform_code_review.asp" method="post" name="codereview"
|
|
|
134 |
onSubmit=" MM_validateForm(
|
|
|
135 |
'date_of_review','Date of Review','RisDate',
|
|
|
136 |
'time_spent','Time Spent','RisNumber',
|
|
|
137 |
'review_reason','Reason for Review','RmaxLength:4000',
|
|
|
138 |
'files_reviewed','Files Reviewed','RmaxLength:4000',
|
|
|
139 |
'review_results','Review Results','R',
|
|
|
140 |
'issues_raised','Issues Raised','RmaxLength:4000'
|
|
|
141 |
);
|
|
|
142 |
if (review_comments.value!='') {MM_validateForm('review_comments','Comments','RmaxLength:4000');};
|
|
|
143 |
if (rteam_domain_expert.value!='') {MM_validateForm('rteam_domain_expert','Domain Expert','RmaxLength:4000');};
|
|
|
144 |
if (rteam_language_expert.value!='') {MM_validateForm('rteam_language_expert','Language Expert','RmaxLength:4000');};
|
|
|
145 |
if (rteam_peer_developer.value!='') {MM_validateForm('rteam_peer_developer','Peer Expert','RmaxLength:4000');};
|
|
|
146 |
if (rteam_author.value!='') {MM_validateForm('rteam_author','Author','RmaxLength:4000');};
|
|
|
147 |
if (fnc_c_meets_functionality.value!='') {MM_validateForm('fnc_c_meets_functionality','FUNCTIONALITY: Meets required functionality','RmaxLength:4000');};
|
|
|
148 |
if (rbs_c_bound_cond_handled.value!='') {MM_validateForm('rbs_c_bound_cond_handled','ROBUSTNESS: Boundary conditions handled','RmaxLength:4000');};
|
|
|
149 |
if (rbs_c_class_interf_precond.value!='') {MM_validateForm('rbs_c_class_interf_precond','ROBUSTNESS: Class interface preconditions tested','RmaxLength:4000');};
|
|
|
150 |
if (rbs_c_no_unitialised_date.value!='') {MM_validateForm('rbs_c_no_unitialised_date','ROBUSTNESS: No unititalised data','RmaxLength:4000');};
|
|
|
151 |
if (rbs_c_excep_handing.value!='') {MM_validateForm('rbs_c_excep_handing','ROBUSTNESS: Exception handling','RmaxLength:4000');};
|
|
|
152 |
if (rbs_c_resource_mng.value!='') {MM_validateForm('rbs_c_resource_mng','ROBUSTNESS: Resource mgt (db cxn, memory)','RmaxLength:4000');};
|
|
|
153 |
if (rbs_c_trans_complet.value!='') {MM_validateForm('rbs_c_trans_complet','ROBUSTNESS: Transaction completeness','RmaxLength:4000');};
|
|
|
154 |
if (rbs_c_threadsafety.value!='') {MM_validateForm('rbs_c_threadsafety','ROBUSTNESS: Threadsafety','RmaxLength:4000');};
|
|
|
155 |
if (rbs_c_ret_vals.value!='') {MM_validateForm('rbs_c_ret_vals','ROBUSTNESS: Return values (where appropriate)','RmaxLength:4000');};
|
|
|
156 |
if (rbs_c_corr_err_handling.value!='') {MM_validateForm('rbs_c_corr_err_handling','ROBUSTNESS: Correct err handling at Subsys bdys','RmaxLength:4000');};
|
|
|
157 |
if (rbs_c_sql_std.value!='') {MM_validateForm('rbs_c_sql_std','ROBUSTNESS: SQL correct and conforming to standards','RmaxLength:4000');};
|
|
|
158 |
if (mnt_c_ext_ref.value!='') {MM_validateForm('mnt_c_ext_ref','MAINTAINABILITY: External references explicit (eg: pkg imports)','RmaxLength:4000');};
|
|
|
159 |
if (mnt_c_class_size.value!='') {MM_validateForm('mnt_c_class_size','MAINTAINABILITY: Class size ok','RmaxLength:4000');};
|
|
|
160 |
if (mnt_c_method_size.value!='') {MM_validateForm('mnt_c_method_size','MAINTAINABILITY: Method size ok','RmaxLength:4000');};
|
|
|
161 |
if (mnt_c_approp_comm.value!='') {MM_validateForm('mnt_c_approp_comm','MAINTAINABILITY: Appropriate commenting','RmaxLength:4000');};
|
|
|
162 |
if (mnt_c_ident_name_std.value!='') {MM_validateForm('mnt_c_ident_name_std','MAINTAINABILITY: Indentifiers conform to naming standards','RmaxLength:4000');};
|
|
|
163 |
if (mnt_c_switch_have_defaults.value!='') {MM_validateForm('mnt_c_switch_have_defaults','MAINTAINABILITY: Switch statements have defaults','RmaxLength:4000');};
|
|
|
164 |
if (mnt_c_no_lit_num.value!='') {MM_validateForm('mnt_c_no_lit_num','MAINTAINABILITY: No literal numeric constants except 1, 0, 1','RmaxLength:4000');};
|
|
|
165 |
if (mnt_c_no_dead_code.value!='') {MM_validateForm('mnt_c_no_dead_code','MAINTAINABILITY: No dead or unused code','RmaxLength:4000');};
|
|
|
166 |
if (dei_c_std_des_patt.value!='') {MM_validateForm('dei_c_std_des_patt','DESIGN ISSUES: Use of standard design patterns','RmaxLength:4000');};
|
|
|
167 |
if (dei_c_approp_algor.value!='') {MM_validateForm('dei_c_approp_algor','DESIGN ISSUES: Use of appropriate algorithms (consider performance)','RmaxLength:4000');};
|
|
|
168 |
if (dei_c_approp_obj.value!='') {MM_validateForm('dei_c_approp_obj','DESIGN ISSUES: Use of appropriate objects (consider bloat, object focus, encapsulation)','RmaxLength:4000');};
|
|
|
169 |
if (dei_c_approp_err_msg.value!='') {MM_validateForm('dei_c_approp_err_msg','DESIGN ISSUES: Appropriate user error messages','RmaxLength:4000');};
|
|
|
170 |
if (dei_c_std_framew_comp.value!='') {MM_validateForm('dei_c_std_framew_comp','DESIGN ISSUES: Use of standard framework components','RmaxLength:4000');};
|
|
|
171 |
if (dei_c_approp_logging.value!='') {MM_validateForm('dei_c_approp_logging','DESIGN ISSUES: Appropriate logging','RmaxLength:4000');};
|
|
|
172 |
return document.MM_returnValue;">
|
|
|
173 |
<table width="100%" border="0" cellspacing="0" cellpadding="2" height="100%">
|
|
|
174 |
<tr>
|
|
|
175 |
<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>
|
|
|
176 |
<td width="1%" background="images/lbox_bg_blue.gif" nowrap class="wform_ttl"> Code Review for <%=pkgName%> <%=pkgVersion%> </td>
|
|
|
177 |
<td width="100%" background="images/lbox_bg_blue.gif" align="right" nowrap class="wform_ttl">
|
|
|
178 |
<input type="submit" name="btn" value="Save" class="form_btn">
|
|
|
179 |
<input type="reset" name="btn" value="Close" class="form_btn" onclick="self.close()">
|
|
|
180 |
</td>
|
|
|
181 |
<td background="images/lbox_bg_blue.gif" align="right" width="1%" nowrap><img src="images/h_trsp_dot.gif" width="5" height="22"></td>
|
|
|
182 |
</tr>
|
|
|
183 |
<tr>
|
|
|
184 |
<td height="100%" width="1%"> </td>
|
|
|
185 |
<td valign="top" nowrap colspan="3" class="wform_ttl" background="images/bg_form_lightgray.gif">
|
|
|
186 |
<br>
|
|
|
187 |
<br>
|
|
|
188 |
<table width="100%" border="0" cellspacing="0" cellpadding="0">
|
|
|
189 |
<tr>
|
|
|
190 |
<td width="1%"><img src="images/h_trsp_dot.gif" width="10" height="30"></td>
|
|
|
191 |
<td width="100%">
|
|
|
192 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
193 |
<tr align="center">
|
|
|
194 |
<td colspan="4" class="form_ttl" background="images/bg_form_lightbluedark.gif">Summary</td>
|
|
|
195 |
</tr>
|
|
|
196 |
<tr>
|
|
|
197 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Date of Review</td>
|
|
|
198 |
<td nowrap bgcolor="#FFFFFF" class="form_item">
|
|
|
199 |
<input name="date_of_review" type="text" size="20" maxlength="20" class="form_item" value='<%If IsNull(rsQry("date_of_review")) Then%><%=EuroDate ( Date )%><%Else%><%=EuroDate ( rsQry("date_of_review") )%><%End If%>'> </td>
|
|
|
200 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field"> Time Spent (hrs)</td>
|
|
|
201 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
202 |
<input name="time_spent" type="text" size="20" maxlength="20" class="form_item" value="<%=rsQry("time_spent")%>"></td>
|
|
|
203 |
</tr>
|
|
|
204 |
<tr>
|
|
|
205 |
<td align="right" valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Reason for Review</td>
|
|
|
206 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item">
|
|
|
207 |
<textarea name="review_reason" cols="80" rows="5" class="form_item"><%=rsQry("review_reason")%></textarea></td>
|
|
|
208 |
</tr>
|
|
|
209 |
<tr>
|
|
|
210 |
<td rowspan="4" align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Review Team </td>
|
|
|
211 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Domain Expert </td>
|
|
|
212 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item">
|
|
|
213 |
<textarea name="rteam_domain_expert" cols="60" rows="2" class="form_item"><%=rsQry("rteam_domain_expert")%></textarea></td>
|
|
|
214 |
</tr>
|
|
|
215 |
<tr>
|
|
|
216 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Language Expert </td>
|
|
|
217 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item">
|
|
|
218 |
<textarea name="rteam_language_expert" cols="60" rows="2" class="form_item"><%=rsQry("rteam_language_expert")%></textarea></td>
|
|
|
219 |
</tr>
|
|
|
220 |
<tr>
|
|
|
221 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Peer Developer </td>
|
|
|
222 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item">
|
|
|
223 |
<textarea name="rteam_peer_developer" cols="60" rows="2" class="form_item"><%=rsQry("rteam_peer_developer")%></textarea></td>
|
|
|
224 |
</tr>
|
|
|
225 |
<tr>
|
|
|
226 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Author</td>
|
|
|
227 |
<td colspan="2" bgcolor="#FFFFFF" class="form_item">
|
|
|
228 |
<textarea name="rteam_author" cols="60" rows="2" class="form_item"><%=rsQry("rteam_author")%></textarea></td>
|
|
|
229 |
</tr>
|
|
|
230 |
<tr>
|
|
|
231 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Files Reviewed</td>
|
|
|
232 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item">
|
|
|
233 |
<textarea name="files_reviewed" cols="80" rows="5" class="form_item"><%=rsQry("files_reviewed")%></textarea></td>
|
|
|
234 |
</tr>
|
|
|
235 |
<tr>
|
|
|
236 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Review Results </td>
|
|
|
237 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item">
|
|
|
238 |
<select name="review_results" class="form_item" id="review_results">
|
|
|
239 |
<%=DrawOverallResult ( rsQry("review_results") )%>
|
|
|
240 |
</select>
|
|
|
241 |
</td>
|
|
|
242 |
</tr>
|
|
|
243 |
<tr>
|
|
|
244 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Issues Raised </td>
|
|
|
245 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item">
|
|
|
246 |
<textarea name="issues_raised" cols="80" rows="5" class="form_item"><%=rsQry("issues_raised")%></textarea></td>
|
|
|
247 |
</tr>
|
|
|
248 |
<tr>
|
|
|
249 |
<td align="right" nowrap background="images/bg_form_lightbluedark.gif" class="form_field">Comments</td>
|
|
|
250 |
<td colspan="3" bgcolor="#FFFFFF" class="form_item">
|
|
|
251 |
<textarea name="review_comments" cols="80" rows="5" class="form_item"><%=rsQry("review_comments")%></textarea></td>
|
|
|
252 |
</tr>
|
|
|
253 |
</table>
|
|
|
254 |
<br>
|
|
|
255 |
<br>
|
|
|
256 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
257 |
<tr align="center">
|
|
|
258 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Functionality</td>
|
|
|
259 |
</tr>
|
|
|
260 |
<tr>
|
|
|
261 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
262 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
263 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
264 |
</tr>
|
|
|
265 |
<tr>
|
|
|
266 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Meets required functionality </td>
|
|
|
267 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
268 |
<select name="fnc_s_meets_functionality" class="form_item" id="fnc_s_meets_functionality">
|
|
|
269 |
<%=DrawItemResult( rsQry("fnc_s_meets_functionality") )%>
|
|
|
270 |
</select>
|
|
|
271 |
</td>
|
|
|
272 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
273 |
<textarea name="fnc_c_meets_functionality" cols="60" rows="3" class="form_item"><%=rsQry("fnc_c_meets_functionality")%></textarea></td>
|
|
|
274 |
</tr>
|
|
|
275 |
</table>
|
|
|
276 |
<br>
|
|
|
277 |
<br>
|
|
|
278 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
279 |
<tr align="center">
|
|
|
280 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Robustness</td>
|
|
|
281 |
</tr>
|
|
|
282 |
<tr>
|
|
|
283 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
284 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
285 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
286 |
</tr>
|
|
|
287 |
<tr>
|
|
|
288 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Boundary conditions handled </td>
|
|
|
289 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
290 |
<select name="rbs_s_bound_cond_handled" class="form_item" id="rbs_s_bound_cond_handled">
|
|
|
291 |
<%=DrawItemResult( rsQry("rbs_s_bound_cond_handled") )%>
|
|
|
292 |
</select>
|
|
|
293 |
</td>
|
|
|
294 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
295 |
<textarea name="rbs_c_bound_cond_handled" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_bound_cond_handled")%></textarea></td>
|
|
|
296 |
</tr>
|
|
|
297 |
<tr>
|
|
|
298 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Class interface preconditions tested</td>
|
|
|
299 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
300 |
<select name="rbs_s_class_interf_precond" class="form_item" id="rbs_s_class_interf_precond">
|
|
|
301 |
<%=DrawItemResult( rsQry("rbs_s_class_interf_precond") )%>
|
|
|
302 |
</select>
|
|
|
303 |
</td>
|
|
|
304 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
305 |
<textarea name="rbs_c_class_interf_precond" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_class_interf_precond")%></textarea></td>
|
|
|
306 |
</tr>
|
|
|
307 |
<tr>
|
|
|
308 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">No unititalised data</td>
|
|
|
309 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
310 |
<select name="rbs_s_no_unitialised_date" class="form_item" id="rbs_s_no_unitialised_date">
|
|
|
311 |
<%=DrawItemResult( rsQry("rbs_s_no_unitialised_date") )%>
|
|
|
312 |
</select>
|
|
|
313 |
</td>
|
|
|
314 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
315 |
<textarea name="rbs_c_no_unitialised_date" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_no_unitialised_date")%></textarea></td>
|
|
|
316 |
</tr>
|
|
|
317 |
<tr>
|
|
|
318 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Exception handling</td>
|
|
|
319 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
320 |
<select name="rbs_s_excep_handing" class="form_item" id="rbs_s_excep_handing">
|
|
|
321 |
<%=DrawItemResult( rsQry("rbs_s_excep_handing") )%>
|
|
|
322 |
</select>
|
|
|
323 |
</td>
|
|
|
324 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
325 |
<textarea name="rbs_c_excep_handing" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_excep_handing")%></textarea></td>
|
|
|
326 |
</tr>
|
|
|
327 |
<tr>
|
|
|
328 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Resource mgt (db cxn, memory)</td>
|
|
|
329 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
330 |
<select name="rbs_s_resource_mng" class="form_item" id="rbs_s_resource_mng">
|
|
|
331 |
<%=DrawItemResult( rsQry("rbs_s_resource_mng") )%>
|
|
|
332 |
</select>
|
|
|
333 |
</td>
|
|
|
334 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
335 |
<textarea name="rbs_c_resource_mng" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_resource_mng")%></textarea></td>
|
|
|
336 |
</tr>
|
|
|
337 |
<tr>
|
|
|
338 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Transaction completeness</td>
|
|
|
339 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
340 |
<select name="rbs_s_trans_complet" class="form_item" id="rbs_s_trans_complet">
|
|
|
341 |
<%=DrawItemResult( rsQry("rbs_s_trans_complet") )%>
|
|
|
342 |
</select>
|
|
|
343 |
</td>
|
|
|
344 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
345 |
<textarea name="rbs_c_trans_complet" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_trans_complet")%></textarea></td>
|
|
|
346 |
</tr>
|
|
|
347 |
<tr>
|
|
|
348 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Threadsafety</td>
|
|
|
349 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
350 |
<select name="rbs_s_threadsafety" class="form_item" id="rbs_s_threadsafety">
|
|
|
351 |
<%=DrawItemResult( rsQry("rbs_s_threadsafety") )%>
|
|
|
352 |
</select>
|
|
|
353 |
</td>
|
|
|
354 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
355 |
<textarea name="rbs_c_threadsafety" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_threadsafety")%></textarea></td>
|
|
|
356 |
</tr>
|
|
|
357 |
<tr>
|
|
|
358 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Return values (where appropriate)</td>
|
|
|
359 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
360 |
<select name="rbs_s_ret_vals" class="form_item" id="rbs_s_ret_vals">
|
|
|
361 |
<%=DrawItemResult( rsQry("rbs_s_ret_vals") )%>
|
|
|
362 |
</select>
|
|
|
363 |
</td>
|
|
|
364 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
365 |
<textarea name="rbs_c_ret_vals" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_ret_vals")%></textarea></td>
|
|
|
366 |
</tr>
|
|
|
367 |
<tr>
|
|
|
368 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Correct err handling at Subsys bdys</td>
|
|
|
369 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
370 |
<select name="rbs_s_corr_err_handling" class="form_item" id="rbs_s_corr_err_handling">
|
|
|
371 |
<%=DrawItemResult( rsQry("rbs_s_corr_err_handling") )%>
|
|
|
372 |
</select>
|
|
|
373 |
</td>
|
|
|
374 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
375 |
<textarea name="rbs_c_corr_err_handling" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_corr_err_handling")%></textarea></td>
|
|
|
376 |
</tr>
|
|
|
377 |
<tr>
|
|
|
378 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">SQL correct and conforming to standards</td>
|
|
|
379 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
380 |
<select name="rbs_s_sql_std" class="form_item" id="rbs_s_sql_std">
|
|
|
381 |
<%=DrawItemResult( rsQry("rbs_s_sql_std") )%>
|
|
|
382 |
</select>
|
|
|
383 |
</td>
|
|
|
384 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
385 |
<textarea name="rbs_c_sql_std" cols="60" rows="3" class="form_item"><%=rsQry("rbs_c_sql_std")%></textarea></td>
|
|
|
386 |
</tr>
|
|
|
387 |
</table>
|
|
|
388 |
<br>
|
|
|
389 |
<br>
|
|
|
390 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
391 |
<tr align="center">
|
|
|
392 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Maintainability</td>
|
|
|
393 |
</tr>
|
|
|
394 |
<tr>
|
|
|
395 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
396 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
397 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
398 |
</tr>
|
|
|
399 |
<tr>
|
|
|
400 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">External references explicit (eg: pkg imports)</td>
|
|
|
401 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
402 |
<select name="mnt_s_ext_ref" class="form_item" id="mnt_s_ext_ref">
|
|
|
403 |
<%=DrawItemResult( rsQry("mnt_s_ext_ref") )%>
|
|
|
404 |
</select>
|
|
|
405 |
</td>
|
|
|
406 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
407 |
<textarea name="mnt_c_ext_ref" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_ext_ref")%></textarea></td>
|
|
|
408 |
</tr>
|
|
|
409 |
<tr>
|
|
|
410 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Class size ok</td>
|
|
|
411 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
412 |
<select name="mnt_s_class_size" class="form_item" id="mnt_s_class_size">
|
|
|
413 |
<%=DrawItemResult( rsQry("mnt_s_class_size") )%>
|
|
|
414 |
</select>
|
|
|
415 |
</td>
|
|
|
416 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
417 |
<textarea name="mnt_c_class_size" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_class_size")%></textarea></td>
|
|
|
418 |
</tr>
|
|
|
419 |
<tr>
|
|
|
420 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Method size ok</td>
|
|
|
421 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
422 |
<select name="mnt_s_method_size" class="form_item" id="mnt_s_method_size">
|
|
|
423 |
<%=DrawItemResult( rsQry("mnt_s_method_size") )%>
|
|
|
424 |
</select>
|
|
|
425 |
</td>
|
|
|
426 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
427 |
<textarea name="mnt_c_method_size" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_method_size")%></textarea></td>
|
|
|
428 |
</tr>
|
|
|
429 |
<tr>
|
|
|
430 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Appropriate commenting</td>
|
|
|
431 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
432 |
<select name="mnt_s_approp_comm" class="form_item" id="mnt_s_approp_comm">
|
|
|
433 |
<%=DrawItemResult( rsQry("mnt_s_approp_comm") )%>
|
|
|
434 |
</select>
|
|
|
435 |
</td>
|
|
|
436 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
437 |
<textarea name="mnt_c_approp_comm" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_approp_comm")%></textarea></td>
|
|
|
438 |
</tr>
|
|
|
439 |
<tr>
|
|
|
440 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Indentifiers conform to naming standards</td>
|
|
|
441 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
442 |
<select name="mnt_s_ident_name_std" class="form_item" id="mnt_s_ident_name_std">
|
|
|
443 |
<%=DrawItemResult( rsQry("mnt_s_ident_name_std") )%>
|
|
|
444 |
</select>
|
|
|
445 |
</td>
|
|
|
446 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
447 |
<textarea name="mnt_c_ident_name_std" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_ident_name_std")%></textarea></td>
|
|
|
448 |
</tr>
|
|
|
449 |
<tr>
|
|
|
450 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Switch statements have defaults</td>
|
|
|
451 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
452 |
<select name="mnt_s_switch_have_defaults" class="form_item" id="mnt_s_switch_have_defaults">
|
|
|
453 |
<%=DrawItemResult( rsQry("mnt_s_switch_have_defaults") )%>
|
|
|
454 |
</select>
|
|
|
455 |
</td>
|
|
|
456 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
457 |
<textarea name="mnt_c_switch_have_defaults" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_switch_have_defaults")%></textarea></td>
|
|
|
458 |
</tr>
|
|
|
459 |
<tr>
|
|
|
460 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">No literal numeric constants except 1, 0, 1</td>
|
|
|
461 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
462 |
<select name="mnt_s_no_lit_num" class="form_item" id="mnt_s_no_lit_num">
|
|
|
463 |
<%=DrawItemResult( rsQry("mnt_s_no_lit_num") )%>
|
|
|
464 |
</select>
|
|
|
465 |
</td>
|
|
|
466 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
467 |
<textarea name="mnt_c_no_lit_num" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_no_lit_num")%></textarea></td>
|
|
|
468 |
</tr>
|
|
|
469 |
<tr>
|
|
|
470 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">No dead or unused code</td>
|
|
|
471 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
472 |
<select name="mnt_s_no_dead_code" class="form_item" id="mnt_s_no_dead_code">
|
|
|
473 |
<%=DrawItemResult( rsQry("mnt_s_no_dead_code") )%>
|
|
|
474 |
</select>
|
|
|
475 |
</td>
|
|
|
476 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
477 |
<textarea name="mnt_c_no_dead_code" cols="60" rows="3" class="form_item"><%=rsQry("mnt_c_no_dead_code")%></textarea></td>
|
|
|
478 |
</tr>
|
|
|
479 |
</table>
|
|
|
480 |
<br>
|
|
|
481 |
<br>
|
|
|
482 |
<table width="100%" border="0" cellspacing="1" cellpadding="4">
|
|
|
483 |
<tr align="center">
|
|
|
484 |
<td colspan="3" class="form_ttl" background="images/bg_form_lightbluedark.gif">Design Issues </td>
|
|
|
485 |
</tr>
|
|
|
486 |
<tr>
|
|
|
487 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Item</td>
|
|
|
488 |
<td width="1%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Check</td>
|
|
|
489 |
<td width="100%" align="center" background="images/bg_form_lightbluedark.gif" class="form_field">Comment</td>
|
|
|
490 |
</tr>
|
|
|
491 |
<tr>
|
|
|
492 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Use of standard design patterns</td>
|
|
|
493 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
494 |
<select name="dei_s_std_des_patt" class="form_item" id="dei_s_std_des_patt">
|
|
|
495 |
<%=DrawItemResult( rsQry("dei_s_std_des_patt") )%>
|
|
|
496 |
</select>
|
|
|
497 |
</td>
|
|
|
498 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
499 |
<textarea name="dei_c_std_des_patt" cols="60" rows="3" class="form_item"><%=rsQry("dei_c_std_des_patt")%></textarea></td>
|
|
|
500 |
</tr>
|
|
|
501 |
<tr>
|
|
|
502 |
<td valign="top" background="images/bg_form_lightbluedark.gif" class="form_item">Use of appropriate algorithms (consider performance)</td>
|
|
|
503 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
504 |
<select name="dei_s_approp_algor" class="form_item" id="dei_s_approp_algor">
|
|
|
505 |
<%=DrawItemResult( rsQry("dei_s_approp_algor") )%>
|
|
|
506 |
</select>
|
|
|
507 |
</td>
|
|
|
508 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
509 |
<textarea name="dei_c_approp_algor" cols="60" rows="3" class="form_item"><%=rsQry("dei_c_approp_algor")%></textarea></td>
|
|
|
510 |
</tr>
|
|
|
511 |
<tr>
|
|
|
512 |
<td valign="top" background="images/bg_form_lightbluedark.gif" class="form_item">Use of appropriate objects (consider bloat, object focus, encapsulation)</td>
|
|
|
513 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
514 |
<select name="dei_s_approp_obj" class="form_item" id="dei_s_approp_obj">
|
|
|
515 |
<%=DrawItemResult( rsQry("dei_s_approp_obj") )%>
|
|
|
516 |
</select>
|
|
|
517 |
</td>
|
|
|
518 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
519 |
<textarea name="dei_c_approp_obj" cols="60" rows="3" class="form_item"><%=rsQry("dei_c_approp_obj")%></textarea></td>
|
|
|
520 |
</tr>
|
|
|
521 |
<tr>
|
|
|
522 |
<td valign="top" background="images/bg_form_lightbluedark.gif" class="form_item">Appropriate user error messages</td>
|
|
|
523 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
524 |
<select name="dei_s_approp_err_msg" class="form_item" id="dei_s_approp_err_msg">
|
|
|
525 |
<%=DrawItemResult( rsQry("dei_s_approp_err_msg") )%>
|
|
|
526 |
</select>
|
|
|
527 |
</td>
|
|
|
528 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
529 |
<textarea name="dei_c_approp_err_msg" cols="60" rows="3" class="form_item"><%=rsQry("dei_c_approp_err_msg")%></textarea></td>
|
|
|
530 |
</tr>
|
|
|
531 |
<tr>
|
|
|
532 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Use of standard framework components</td>
|
|
|
533 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
534 |
<select name="dei_s_std_framew_comp" class="form_item" id="dei_s_std_framew_comp">
|
|
|
535 |
<%=DrawItemResult( rsQry("dei_s_std_framew_comp") )%>
|
|
|
536 |
</select>
|
|
|
537 |
</td>
|
|
|
538 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
539 |
<textarea name="dei_c_std_framew_comp" cols="60" rows="3" class="form_item"><%=rsQry("dei_c_std_framew_comp")%></textarea></td>
|
|
|
540 |
</tr>
|
|
|
541 |
<tr>
|
|
|
542 |
<td valign="top" nowrap background="images/bg_form_lightbluedark.gif" class="form_item">Appropriate logging</td>
|
|
|
543 |
<td valign="top" bgcolor="#FFFFFF">
|
|
|
544 |
<select name="dei_s_approp_logging" class="form_item" id="dei_s_approp_logging">
|
|
|
545 |
<%=DrawItemResult( rsQry("dei_s_approp_logging") )%>
|
|
|
546 |
</select>
|
|
|
547 |
</td>
|
|
|
548 |
<td bgcolor="#FFFFFF" class="form_item">
|
|
|
549 |
<textarea name="dei_c_approp_logging" cols="60" rows="3" class="form_item"><%=rsQry("dei_c_approp_logging")%></textarea></td>
|
|
|
550 |
</tr>
|
|
|
551 |
</table>
|
|
|
552 |
<br>
|
|
|
553 |
<br>
|
|
|
554 |
</td>
|
|
|
555 |
</tr>
|
|
|
556 |
</table>
|
|
|
557 |
</td>
|
|
|
558 |
</tr>
|
|
|
559 |
<tr>
|
|
|
560 |
<td height="1%" width="1%"><img src="images/h_trsp_dot.gif" width="5" height="5"></td>
|
|
|
561 |
<td valign="top" nowrap colspan="3" class="wform_ttl" background="images/lbox_bg_blue.gif" align="right">
|
|
|
562 |
<input type="submit" name="btn" value="Save" class="form_btn">
|
|
|
563 |
<input type="reset" name="btn" value="Close" class="form_btn" onclick="self.close()">
|
|
|
564 |
</td>
|
|
|
565 |
</tr>
|
|
|
566 |
</table>
|
|
|
567 |
<input type="hidden" name="action" value="true">
|
|
|
568 |
<input type="hidden" name="pv_id" value="<%=parPv_id%>">
|
|
|
569 |
<input type="hidden" name="rtag_id" value="<%=parRtag_id%>">
|
|
|
570 |
</form>
|
|
|
571 |
</body>
|
|
|
572 |
</html>
|
|
|
573 |
<%rsQry.Close
|
|
|
574 |
Set rsQry = Nothing%>
|
|
|
575 |
<!-- DESTRUCTOR ------->
|
|
|
576 |
<!--#include file="common/destructor.asp"-->
|