Copy globalVarScopedMap entry on PlayerCharacter clone

PlayerCharacter.clone() invokes bean.copyContents(id, aClone.id) on every
facet, but AbstractItemFacet.copyContents() copies the underlying map
directly without going through set(), so ScopeFacet.set()'s side effect
of registering the GlobalPCVarScoped sentinel was skipped on the clone.

The cloned character then had a null entry in globalVarScopedMap, and
getGlobalScope() handed back a ScopeInstance whose owner was null,
causing NullPointerExceptions in VarScoped.getProviderFor() during the
26 pcGenGUI*Test save/restore flows.

Override copyContents() to also copy the globalVarScopedMap entry.

Author: Vest

code/src/java/pcgen/cdom/facet/ScopeFacet.java

@@ -45,6 +45,17 @@ public boolean set(CharID id, ScopeInstanceFactory factory)
 		return result;
 	}
 
+	@Override
+	public void copyContents(CharID source, CharID copy)
+	{
+		super.copyContents(source, copy);
+		GlobalPCVarScoped globalVS = globalVarScopedMap.get(source);
+		if (globalVS != null)
+		{
+			globalVarScopedMap.put(copy, globalVS);
+		}
+	}
+
 	@Override
 	public ScopeInstanceFactory remove(CharID id)
 	{