Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to delete Microschemas #192

Closed
deckdom opened this issue Nov 9, 2017 · 2 comments
Closed

Unable to delete Microschemas #192

deckdom opened this issue Nov 9, 2017 · 2 comments
Labels

Comments

@deckdom
Copy link
Member

deckdom commented Nov 9, 2017

Gentics Mesh Version, operating system, or hardware.

  • v0.10.4

Operating System

  • Linux
  • MacOSX
  • Windows

JVM

  • Open JDK 1.8.0_151

Problem

There's a prevention from deleting a microschema if it is still in use in a node. However, this seems like it's not updating properly, as when I delete all nodes with such a microschema, I still get an error when attempting to delete it:

{
	"i18nParameters": [
		"8c5cee4596f744469cee4596f7244689"
	],
	"properties": {
	},
	"type": "generic_error",
	"message": "There are micronodes which use the microschema 8c5cee4596f744469cee4596f7244689. Delete the nodes which use the micronodes first."
}

I also noticed that sometimes a restart can help, but not always.

Reproducer

  • Create a Microschema and a Schema which has a reference to the new microschema
  • Create a new node with the newly created schema
  • Delete the node
  • Attempt to delete the microschema
  • You get an error as described above

Expected behaviour and actual behaviour

That it's possible to delete it without having to restart the server

@deckdom deckdom added the bug label Nov 9, 2017
@Jotschi
Copy link
Contributor

Jotschi commented Nov 9, 2017

@SiNEXiST It would be really helpful if you could add a Unit test for that usecase.

You could actually extend the MicroschemaEndpointTest#testDeleteByUUIDWhileInUse testcase.

@Jotschi
Copy link
Contributor

Jotschi commented Jun 12, 2019

Test is now green since we are now correctly deleting micronode fields and micronode list fields.

@Jotschi Jotschi closed this as completed Jun 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants