Fix improper usage of 'dump' bitmap

Now that 'dump' is a bitmap, we can't simply set it to 'true'.

Noticed while debugging the prior issue.

Author: sfrost

src/bin/pg_dump/pg_dump_sort.c

@@ -848,9 +848,14 @@ repairTypeFuncLoop(DumpableObject *typeobj, DumpableObject *funcobj)
 	if (typeInfo->shellType)
 	{
 		addObjectDependency(funcobj, typeInfo->shellType->dobj.dumpId);
-		/* Mark shell type as to be dumped if any such function is */
+		/*
+		 * Mark shell type (always including the definition, as we need
+		 * the shell type defined to identify the function fully) as to be
+		 * dumped if any such function is
+		 */
 		if (funcobj->dump)
-			typeInfo->shellType->dobj.dump = true;
+			typeInfo->shellType->dobj.dump = funcobj->dump |
+											 DUMP_COMPONENT_DEFINITION;
 	}
 }