diff --git a/5.Code Inspection/CodeInspection.lyx b/5.Code Inspection/CodeInspection.lyx index b2dbcad..5c3b4bb 100644 --- a/5.Code Inspection/CodeInspection.lyx +++ b/5.Code Inspection/CodeInspection.lyx @@ -251,7 +251,7 @@ end{minipage} \begin_layout Standard \begin_inset Graphics - filename /home/giulio/gitstuff/private/swing2/3.Integration Test Plan Document/res/img/logopm.pdf + filename C:/Users/Giancarlo_Colaci/Desktop/swing2/1.RASD/res/img/logopm.pdf width 30text% \end_inset @@ -507,7 +507,7 @@ Issues \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -582,7 +582,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -591,7 +591,15 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +L136: The method +\shape italic +seoUrl +\shape default + should be called +\shape italic +getTransformedSeoUrl +\shape default +in order to better explain the performed action \end_layout \end_inset @@ -600,7 +608,15 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +L138: The method +\shape italic +seoUrl +\shape default + should be called +\shape italic +getTransformedSeoUrl +\shape default +in order to better explain the performed action \end_layout \end_inset @@ -622,7 +638,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +None \end_layout \end_inset @@ -631,7 +647,7 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +None \end_layout \end_inset @@ -640,7 +656,7 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +None \end_layout \end_inset @@ -662,7 +678,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.22 +None \end_layout \end_inset @@ -671,7 +687,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.09 +None \end_layout \end_inset @@ -680,7 +696,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.00 +RegExp \end_layout \end_inset @@ -702,7 +718,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -711,7 +727,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -720,7 +736,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -742,7 +758,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -751,7 +767,15 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +L136: The method +\shape italic +seoUrl +\shape default + should be called +\shape italic +getTransformedSeoUrl +\shape default +because a method should always start with a verb \end_layout \end_inset @@ -760,7 +784,15 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +L138: The method +\shape italic +seoUrl +\shape default + should be called +\shape italic +getTransformedSeoUrl +\shape default +because a method should always start with a verb \end_layout \end_inset @@ -782,7 +814,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -791,7 +823,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -800,7 +832,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -822,7 +854,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -831,7 +863,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -840,7 +872,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout - +None \end_layout \end_inset @@ -875,7 +907,7 @@ Naming Conventions \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -950,7 +982,12 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +L169-174: They split the +\shape italic +productConfigItemContent +\shape default + definition into 6 different lines (adding 8 spaces at the beginning of + each line) in order to obtain a more readable instruction \end_layout \end_inset @@ -959,7 +996,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -968,7 +1005,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -990,7 +1027,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +None \end_layout \end_inset @@ -999,7 +1036,7 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +None \end_layout \end_inset @@ -1008,7 +1045,7 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +None \end_layout \end_inset @@ -1043,7 +1080,7 @@ Indention \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -1118,7 +1155,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -1127,7 +1164,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1136,7 +1173,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1158,7 +1195,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +None \end_layout \end_inset @@ -1167,7 +1204,7 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +None \end_layout \end_inset @@ -1176,7 +1213,7 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +None \end_layout \end_inset @@ -1211,7 +1248,7 @@ Braces \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -1286,7 +1323,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -1295,7 +1332,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1304,7 +1341,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1326,7 +1363,13 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +L55 (89), L57 (131 without comment, 181 with it), L68 (148), L72 (139), + L82 (105), L89 (139), L92 (81), L93 (188), L105 (99), L110 (171), L111 + (91), L112 (159), L113 (192), L116 (186), L117 (137), L120 (226), L122 + (86), L123 (197), L130 (156), L133 (146), L142 (102), L143 (152), L146 + (102), L147 (152), L152 (280), L153 (159), L156 (295), L166 (133), L169 + (103), L170 (101), L176 (153 only comment), L179 (81), L180 (180), L184 + (86), L185 (92), L188 (145), L191 (89) \end_layout \end_inset @@ -1335,7 +1378,9 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +L88 (98), L89 (102), L92 (85), L95 (99), L98 (107), L102 (100), L103 (136), + L106 (82), L138 (114), L139 (88), L154 (104), L155 (109), L166 (82), L169 + (102), L178 (103) \end_layout \end_inset @@ -1344,7 +1389,10 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +L88 (98), L89 (102), L92 (85), L95 (99), L98 (107), L102 (100), L103 (159), + L106 (82), L140 (114), L141 (88), L156 (104), L157 (109), L168 (82), L171 + (102), L180 (103), L191 (113) (only comment), L197 (81), L201 (141), L202 + (88), L206 (86), L209 (102), L213 (88) \end_layout \end_inset @@ -1366,7 +1414,11 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.22 +L57 (131 with comment, 181 without it), L68 (148), L72 (139), L89 (139), + L93 (188), L110 (171), L112 (159), L113 (192), L116 (186), L117 (137), + L120 (226), L123 (197), L130 (156), L133 (146), L143 (152), L147 (152), + L152 (280), L153 (159), L156 (295), L166 (133),L176 (153 only comment), + L180 (180), L188 (145) \end_layout \end_inset @@ -1375,7 +1427,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.09 +L103 (136) \end_layout \end_inset @@ -1384,7 +1436,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.00 +L103 (159), L201 (141) \end_layout \end_inset @@ -1419,7 +1471,7 @@ File Organization \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -1494,7 +1546,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -1503,7 +1555,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1512,7 +1564,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1534,7 +1586,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +None \end_layout \end_inset @@ -1543,7 +1595,7 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +None \end_layout \end_inset @@ -1552,7 +1604,7 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +None \end_layout \end_inset @@ -1574,7 +1626,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.22 +None \end_layout \end_inset @@ -1583,7 +1635,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.09 +None \end_layout \end_inset @@ -1592,7 +1644,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.00 +None \end_layout \end_inset @@ -1702,7 +1754,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +Very few comments about class, interface, methods, and blocks of code \end_layout \end_inset @@ -1711,7 +1763,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +Very few comments about class, interface, methods, and blocks of code \end_layout \end_inset @@ -1720,7 +1772,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +Very few comments about class, interface, methods, and blocks of code \end_layout \end_inset @@ -1742,7 +1794,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +None \end_layout \end_inset @@ -1751,7 +1803,7 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +None \end_layout \end_inset @@ -1760,7 +1812,7 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +None \end_layout \end_inset @@ -1795,7 +1847,7 @@ Comments \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -1870,7 +1922,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -1879,7 +1931,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1888,7 +1940,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -1910,7 +1962,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Some phone, mail +None \end_layout \end_inset @@ -1919,7 +1971,7 @@ Some phone, mail \begin_inset Text \begin_layout Plain Layout -Individual phone, fax +None \end_layout \end_inset @@ -1928,7 +1980,7 @@ Individual phone, fax \begin_inset Text \begin_layout Plain Layout -Narrow band email +None \end_layout \end_inset @@ -1950,7 +2002,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.22 + \end_layout \end_inset @@ -1959,7 +2011,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.09 + \end_layout \end_inset @@ -1968,7 +2020,7 @@ Narrow band email \begin_inset Text \begin_layout Plain Layout -1.00 + \end_layout \end_inset @@ -2118,7 +2170,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -2127,7 +2179,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -2136,7 +2188,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -3123,7 +3175,7 @@ Arrays \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular @@ -3198,7 +3250,7 @@ UrlRegexpTransform \begin_inset Text \begin_layout Plain Layout -International +None \end_layout \end_inset @@ -3207,7 +3259,7 @@ International \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -3216,7 +3268,7 @@ Multi-city and multi-company \begin_inset Text \begin_layout Plain Layout -Multi-city and multi-company +None \end_layout \end_inset @@ -3251,7 +3303,7 @@ Object Comparison \begin_inset Float table wide false sideways false -status collapsed +status open \begin_layout Plain Layout \begin_inset Tabular