mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-08-07 17:49:17 +00:00
Fallout from fix ALF-3884: Share does not report access denied exceptions correctly
- The low-level exception was changed to be more explicit about the 'read-only' state git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@22451 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
@@ -30,6 +30,7 @@ import org.alfresco.repo.security.permissions.AccessDeniedException;
|
|||||||
import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
|
import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
|
||||||
import org.alfresco.service.cmr.repository.NodeService;
|
import org.alfresco.service.cmr.repository.NodeService;
|
||||||
import org.alfresco.service.cmr.repository.StoreRef;
|
import org.alfresco.service.cmr.repository.StoreRef;
|
||||||
|
import org.alfresco.service.transaction.ReadOnlyServerException;
|
||||||
import org.alfresco.util.ApplicationContextHelper;
|
import org.alfresco.util.ApplicationContextHelper;
|
||||||
import org.hibernate.dialect.Dialect;
|
import org.hibernate.dialect.Dialect;
|
||||||
import org.hibernate.dialect.PostgreSQLDialect;
|
import org.hibernate.dialect.PostgreSQLDialect;
|
||||||
@@ -141,6 +142,13 @@ public class TransactionServiceImplTest extends TestCase
|
|||||||
txn.commit();
|
txn.commit();
|
||||||
fail("Read-only transaction wasn't detected");
|
fail("Read-only transaction wasn't detected");
|
||||||
}
|
}
|
||||||
|
catch (ReadOnlyServerException e)
|
||||||
|
{
|
||||||
|
// This is now thrown at the lower layers, but it *is* possible for one of the later
|
||||||
|
// exceptions to get through: Fixed ALF-3884: Share does not report access denied exceptions correctly
|
||||||
|
@SuppressWarnings("unused")
|
||||||
|
int i = 0;
|
||||||
|
}
|
||||||
catch (InvalidDataAccessApiUsageException e)
|
catch (InvalidDataAccessApiUsageException e)
|
||||||
{
|
{
|
||||||
// expected this ...
|
// expected this ...
|
||||||
|
Reference in New Issue
Block a user