CCM NG/ccm-cms: Fixed some more test failures

git-svn-id: https://svn.libreccm.org/ccm/ccm_ng@4722 8810af33-2d31-482b-a856-94f89814c4df
jensp 2017-05-10 15:33:34 +00:00
parent bd85950ce5
commit 99333a3886
7 changed files with 28 additions and 37 deletions

View File

@ -362,7 +362,7 @@ public class AssetManager {
index));
}
assetRepo.save(asset);
assetRepo.save(copy);
categoryManager.addObjectToCategory(copy,
targetFolder,

View File

@ -470,7 +470,6 @@ public class AssetManagerTest {
final Folder targetFolder = folderRepo.findById(-420L).get();
assetManager.copy(asset, targetFolder);
assetManager.copy(asset, targetFolder);
}
/**

View File

@ -417,7 +417,7 @@ public class ContentTypeRepositoryTest {
"META-INF/persistence.xml")
.addAsWebInfResource("test-web.xml", "web.xml")
.addAsResource("configs/shiro.ini", "shiro.ini")
.addAsWebInfResource("META-INF/beans.xml", "WEB-INF/beans.xml");
.addAsWebInfResource("META-INF/beans.xml", "beans.xml");
}
/**
@ -651,8 +651,6 @@ public class ContentTypeRepositoryTest {
.execute(() -> contentTypeRepo.delete(newsType.get()));
}
// ToDo, does not work at the moment because including AuthorizationInterceptor
// pulls in to many other dependencies, more exactly most of Bebop...
/**
* Verifies that an unused content type can be deleted.
*/

View File

@ -76,9 +76,9 @@
<ccm_core.ccm_objects object_id="-1800"
display_name="catalog.pdf_copy1"
uuid="00000000-0000-0000-0000-000000000000" />
<ccm_core.ccm_objects object_id="-1900"
<!-- <ccm_core.ccm_objects object_id="-1900"
display_name="catalog.pdf_copy2"
uuid="00000000-0000-0000-0000-000000000001" />
uuid="00000000-0000-0000-0000-000000000001" />-->
<ccm_core.ccm_objects_aud object_id="-600"
rev="0"
@ -112,10 +112,10 @@
rev="1"
revtype="0"
display_name="catalog.pdf_copy1" />
<ccm_core.ccm_objects_aud object_id="-1900"
<!-- <ccm_core.ccm_objects_aud object_id="-1900"
rev="1"
revtype="0"
display_name="catalog.pdf_copy2" />
display_name="catalog.pdf_copy2" />-->
<ccm_core.categories object_id="-200"
unique_id="82014239-9c06-486d-ae8c-4ae47f52a699"
@ -305,7 +305,7 @@
<ccm_cms.assets object_id="-1100" />
<ccm_cms.assets object_id="-1150" />
<ccm_cms.assets object_id="-1800" />
<ccm_cms.assets object_id="-1900" />
<!--<ccm_cms.assets object_id="-1900" />-->
<ccm_cms.assets_aud object_id="-700"
rev="0" />
@ -321,8 +321,8 @@
rev="0" />
<ccm_cms.assets_aud object_id="-1800"
rev="1" />
<ccm_cms.assets_aud object_id="-1900"
rev="1" />
<!-- <ccm_cms.assets_aud object_id="-1900"
rev="1" />-->
<ccm_cms.asset_titles asset_id="-700"
localized_value="header.png"
@ -345,9 +345,9 @@
<ccm_cms.asset_titles asset_id="-1800"
localized_value="catalog.pdf"
locale="en" />
<ccm_cms.asset_titles asset_id="-1900"
<!-- <ccm_cms.asset_titles asset_id="-1900"
localized_value="catalog.pdf"
locale="en" />
locale="en" />-->
<ccm_cms.binary_assets object_id="-700"
filename="header.png"
@ -377,10 +377,10 @@
filename="catalog.pdf"
mime_type="application/pdf"
data_size="0"/>
<ccm_cms.binary_assets object_id="-1900"
<!-- <ccm_cms.binary_assets object_id="-1900"
filename="catalog.pdf"
mime_type="application/pdf"
data_size="0"/>
data_size="0"/>-->
<ccm_cms.binary_assets_aud object_id="-700"
rev="0"
@ -417,11 +417,11 @@
filename="catalog.pdf"
mime_type="application/pdf"
data_size="0" />
<ccm_cms.binary_assets_aud object_id="-1900"
<!-- <ccm_cms.binary_assets_aud object_id="-1900"
rev="1"
filename="catalog.pdf"
mime_type="application/pdf"
data_size="0" />
data_size="0" />-->
<ccm_cms.images object_id="-700"
height="0"
@ -456,7 +456,7 @@
<ccm_cms.files object_id="-1000" />
<ccm_cms.files object_id="-1100" />
<ccm_cms.files object_id="-1800" />
<ccm_cms.files object_id="-1900" />
<!--<ccm_cms.files object_id="-1900" />-->
<ccm_cms.files_aud object_id="-1000"
rev="0" />
@ -464,8 +464,8 @@
rev="0" />
<ccm_cms.files_aud object_id="-1800"
rev="1" />
<ccm_cms.files_aud object_id="-1900"
rev="1" />
<!-- <ccm_cms.files_aud object_id="-1900"
rev="1" />-->
<ccm_cms.asset_titles_aud asset_id="-700"
rev="0"
@ -502,11 +502,11 @@
revtype="0"
localized_value="catalog.pdf"
locale="en" />
<ccm_cms.asset_titles_aud asset_id="-1900"
<!-- <ccm_cms.asset_titles_aud asset_id="-1900"
rev="1"
revtype="0"
localized_value="catalog.pdf"
locale="en" />
locale="en" />-->
<ccm_cms.content_items_aud object_id="-600"
rev="0"
@ -594,13 +594,13 @@
object_order="7"
category_index="false"
type="folder" />
<ccm_core.categorizations categorization_id="-30800"
<!-- <ccm_core.categorizations categorization_id="-30800"
category_id="-420"
object_id="-1900"
category_order="1"
object_order="8"
category_index="false"
type="folder" />
type="folder" />-->
<ccm_cms.attachment_lists list_id="-40100"
name="images"

View File

@ -514,7 +514,7 @@
<ccm_core.categorizations categorization_id="-30400"
category_id="-1600"
object_id="-900"
category_order="2"
category_order="1"
object_order="4"
category_index="false"
type="folder" />

View File

@ -514,7 +514,7 @@
<ccm_core.categorizations categorization_id="-30400"
category_id="-410"
object_id="-900"
category_order="2"
category_order="1"
object_order="4"
category_index="false"
type="folder" />

View File

@ -127,12 +127,6 @@ public class CategoryManager {
Objects.requireNonNull(category,
"Can't add an object to category 'null'.");
final CcmObject addedObject = ccmObjectRepo
.findObjectById(object.getObjectId())
.orElseThrow(() -> new IllegalArgumentException(String.format(
"No CcmObject with ID %d in the database. "
+ "Where did that ID come from?",
object.getObjectId())));
final Category assignedCategory = categoryRepo
.findById(category.getObjectId())
.orElseThrow(() -> new IllegalArgumentException(String.format(
@ -141,17 +135,17 @@ public class CategoryManager {
category.getObjectId())));
final Categorization categorization = new Categorization();
categorization.setCategorizedObject(addedObject);
categorization.setCategorizedObject(object);
categorization.setCategory(assignedCategory);
final long categoryCount = countAssignedCategories(addedObject);
final long categoryCount = countAssignedCategories(object);
categorization.setCategoryOrder(categoryCount + 1);
final long objectCount = countObjects(assignedCategory);
categorization.setObjectOrder(objectCount + 1);
categorization.setType(type);
categorization.setIndex(false);
addedObject.addCategory(categorization);
object.addCategory(categorization);
assignedCategory.addObject(categorization);
// Saving a category requires the manage_category privilege which