I fixed it in 3.2.x and trunk. It did not seem to be present in 3.1.
I'll take a look at 3.0.x today, I don't remember if I checked that
branch or not.
---- Cris J H
On 08/06/2010 08:55 PM, Eric Dalquist wrote:
That is a very good catch and is a bug. Are you able to take care of
getting that fixed? I think it affect trunk, 3.2, 3.1 and 3.0
-Eric
On 8/6/10 1:19 PM, Cris J Holdorph wrote:
Please take a look at uPortal 3.2.2 the file:
uportal-impl/src/main/java/org/jasig/portal/layout/dlm/RDBMDistributedLayoutStore.java
Specifically lines 1893-1903...
They look like this:
final ResultSet rs2 = pstmt2.executeQuery();
try {
while (rs2.next()) {
String pName = rs.getString(1);
if (tsd.containsParameterName(pName))
tsup.putParameterValue(pName, rs2.getString(2));
}
}
finally {
close(rs2);
}
Shouldn't the 'rs.getString(1)' be 'rs2.getString(1)' instead? I'm
changing this and checking it in as a fix to UP-2781. However, I would
like feedback from whomever might have made the changes, to be sure
I'm not missing something.
---- Cris J H
--
You are currently subscribed to [email protected] as:
[email protected]
To unsubscribe, change settings or access archives, see
http://www.ja-sig.org/wiki/display/JSG/uportal-dev
