While reviewing Tara's patch I came across this code in storeObject:

                statement =
                    connection.prepareStatement(
                        "select 1 from OBJECT o, URI u where o.URI_ID=u.URI_ID and 
u.URI_STRING=?");
                statement.setString(1, uri.toString());
                res = statement.executeQuery();
                if (res.next()) {
                    if (create)
                        return false;
                } else {
                    if (!create)
                        return false;
                }

Isn't this nonsense? Maybe it was meant to be:

                if (res.next()) {
                    if (create)
                        return false;
                } else {
                    if (!create)
                        return true;
                }

Does anyone know if this is needed anyhow as it did not seem to cause any problems in the past?


Clueless...

Oliver


--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to