Comment merge methods

[koeaw]

I think it would be helpful to add docstrings, if brief ones, to the merge methods in AbstractEntity.

Specifically:

apis-core-rdf/apis_core/apis_entities/models.py

Line 87 in 1323d33

def get_merge_view_url(self):

and:

apis-core-rdf/apis_core/apis_entities/models.py

Line 100 in 1323d33

def merge_with(self, entities):

The fact the former looks like it mirrors the duplicate method's implementation but not its name pattern throws an additional wrench into trying to infer what's what a glance.

apis-core-rdf/apis_core/apis_entities/models.py

Line 80 in 1323d33

def get_duplicate_url(self):

[koeaw]

Ok, the duplicate method's name seems to follow the pattern of most methods in GenericModel, which also has some with "view" in their names.

apis-core-rdf/apis_core/generic/abc.py

Lines 20 to 30 in 1323d33

	def get_importview_url(cls):
	ct = ContentType.objects.get_for_model(cls)
	return reverse("apis_core:generic:import", args=[ct])
	def get_edit_url(self):
	ct = ContentType.objects.get_for_model(self)
	return reverse("apis_core:generic:update", args=[ct, self.id])
	def get_enrich_url(self):
	ct = ContentType.objects.get_for_model(self)
	return reverse("apis_core:generic:enrich", args=[ct, self.id])

Not sure if the word is meant to differentiate between different types of actions/views, but what I said above about it adding an element of confusion remains unchanged.