child_of function refactoring

[alanking]

This function could use some work...

irods_capability_automated_ingest/irods_capability_automated_ingest/sync_irods.py

Lines 25 to 43 in dfbe556

	def child_of(session, child_resc_name, resc_name):
	if child_resc_name == resc_name:
	return True
	else:
	while True:
	child_resc = session.resources.get(child_resc_name)
	parent_resc_id = child_resc.parent
	if parent_resc_id is None:
	break
	parent_resc_name = None
	for row in session.query(Resource.name).filter(
	Resource.id == parent_resc_id
	):
	parent_resc_name = row[Resource.name]
	if parent_resc_name == resc_name:
	return True
	child_resc_name = parent_resc_name
	return False

• else can be removed and reduce indentation by one for most of the function
• The name could be better
• The embedded for loop over the parent_id query is confusing