From 2f917fd7c00ba4f80d85d2908781b806c77d8ce9 Mon Sep 17 00:00:00 2001
From: lunarcleint <88809950+lunarcleint@users.noreply.github.com>
Date: Wed, 12 Jun 2024 03:12:50 -0700
Subject: [PATCH 01/12] Fix Character isPlayer Bug / Better .xml saving

---
 source/Xml.hx                                 | 410 +++++++++++++
 source/funkin/backend/utils/CoolUtil.hx       |  10 +
 .../editors/character/CharacterInfoScreen.hx  |   3 +-
 source/funkin/game/Character.hx               | 551 +++++++-----------
 source/haxe/xml/Printer.hx                    | 137 +++++
 5 files changed, 781 insertions(+), 330 deletions(-)
 create mode 100644 source/Xml.hx
 create mode 100644 source/haxe/xml/Printer.hx

diff --git a/source/Xml.hx b/source/Xml.hx
new file mode 100644
index 00000000..82613dcd
--- /dev/null
+++ b/source/Xml.hx
@@ -0,0 +1,410 @@
+/*
+ * Copyright (C)2005-2019 Haxe Foundation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+/**
+	Xml node types.
+
+	@see https://haxe.org/manual/std-Xml.html
+**/
+enum abstract XmlType(Int) {
+	/**
+		Represents an XML element type.
+	**/
+	var Element = 0;
+
+	/**
+		Represents XML parsed character data type.
+	**/
+	var PCData = 1;
+
+	/**
+		Represents XML character data type.
+	**/
+	var CData = 2;
+
+	/**
+		Represents an XML comment type.
+	**/
+	var Comment = 3;
+
+	/**
+		Represents an XML doctype element type.
+	**/
+	var DocType = 4;
+
+	/**
+		Represents an XML processing instruction type.
+	**/
+	var ProcessingInstruction = 5;
+
+	/**
+		Represents an XML document type.
+	**/
+	var Document = 6;
+
+	public function toString():String {
+		return switch (cast this : XmlType) {
+			case Element: "Element";
+			case PCData: "PCData";
+			case CData: "CData";
+			case Comment: "Comment";
+			case DocType: "DocType";
+			case ProcessingInstruction: "ProcessingInstruction";
+			case Document: "Document";
+		};
+	}
+}
+
+/**
+	Cross-platform Xml API.
+
+	@see https://haxe.org/manual/std-Xml.html
+**/
+class Xml {
+	/**
+		XML element type.
+	**/
+	static public var Element(default, never) = XmlType.Element;
+
+	/**
+		XML parsed character data type.
+	**/
+	static public var PCData(default, never) = XmlType.PCData;
+
+	/**
+		XML character data type.
+	**/
+	static public var CData(default, never) = XmlType.CData;
+
+	/**
+		XML comment type.
+	**/
+	static public var Comment(default, never) = XmlType.Comment;
+
+	/**
+		XML doctype element type.
+	**/
+	static public var DocType(default, never) = XmlType.DocType;
+
+	/**
+		XML processing instruction type.
+	**/
+	static public var ProcessingInstruction(default, never) = XmlType.ProcessingInstruction;
+
+	/**
+		XML document type.
+	**/
+	static public var Document(default, never) = XmlType.Document;
+
+	/**
+		Parses the String into an Xml document.
+	**/
+	static public function parse(str:String):Xml {
+		return haxe.xml.Parser.parse(str);
+	}
+
+	/**
+		Returns the type of the Xml Node. This should be used before
+		accessing other functions since some might raise an exception
+		if the node type is not correct.
+	**/
+	public var nodeType(default, null):XmlType;
+
+	/**
+		Returns the node name of an Element.
+	**/
+	@:isVar public var nodeName(get, set):String;
+
+	/**
+		Returns the node value. Only works if the Xml node is not an Element or a Document.
+	**/
+	@:isVar public var nodeValue(get, set):String;
+
+	/**
+		Returns the parent object in the Xml hierarchy.
+		The parent can be `null`, an Element or a Document.
+	**/
+	public var parent(default, null):Xml;
+
+	var children:Array<Xml>;
+	var attributeMap:Map<String, String>;
+
+	public var attributeOrder:Array<String>;
+
+	#if !cppia inline #end function get_nodeName() {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		return nodeName;
+	}
+
+	#if !cppia inline #end function set_nodeName(v) {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		return this.nodeName = v;
+	}
+
+	#if !cppia inline #end function get_nodeValue() {
+		if (nodeType == Document || nodeType == Element) {
+			throw 'Bad node type, unexpected $nodeType';
+		}
+		return nodeValue;
+	}
+
+	#if !cppia inline #end function set_nodeValue(v) {
+		if (nodeType == Document || nodeType == Element) {
+			throw 'Bad node type, unexpected $nodeType';
+		}
+		return this.nodeValue = v;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createElement(name:String):Xml {
+		var xml = new Xml(Element);
+		xml.nodeName = name;
+		return xml;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createPCData(data:String):Xml {
+		var xml = new Xml(PCData);
+		xml.nodeValue = data;
+		return xml;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createCData(data:String):Xml {
+		var xml = new Xml(CData);
+		xml.nodeValue = data;
+		return xml;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createComment(data:String):Xml {
+		var xml = new Xml(Comment);
+		xml.nodeValue = data;
+		return xml;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createDocType(data:String):Xml {
+		var xml = new Xml(DocType);
+		xml.nodeValue = data;
+		return xml;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createProcessingInstruction(data:String):Xml {
+		var xml = new Xml(ProcessingInstruction);
+		xml.nodeValue = data;
+		return xml;
+	}
+
+	/**
+		Creates a node of the given type.
+	**/
+	static public function createDocument():Xml {
+		return new Xml(Document);
+	}
+
+	/**
+		Get the given attribute of an Element node. Returns `null` if not found.
+		Attributes are case-sensitive.
+	**/
+	public function get(att:String):String {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		return attributeMap[att];
+	}
+
+	/**
+		Set the given attribute value for an Element node.
+		Attributes are case-sensitive.
+	**/
+	public function set(att:String, value:String):Void {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		attributeMap.set(att, value);
+	}
+
+	/**
+		Removes an attribute for an Element node.
+		Attributes are case-sensitive.
+	**/
+	public function remove(att:String):Void {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		attributeMap.remove(att);
+	}
+
+	/**
+		Tells if the Element node has a given attribute.
+		Attributes are case-sensitive.
+	**/
+	public function exists(att:String):Bool {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		return attributeMap.exists(att);
+	}
+
+	/**
+		Returns an `Iterator` on all the attribute names.
+	**/
+	public function attributes():Iterator<String> {
+		if (nodeType != Element) {
+			throw 'Bad node type, expected Element but found $nodeType';
+		}
+		return attributeMap.keys();
+	}
+
+	/**
+		Returns an iterator of all child nodes.
+		Only works if the current node is an Element or a Document.
+	**/
+	public #if !cppia inline #end function iterator():Iterator<Xml> {
+		ensureElementType();
+		return children.iterator();
+	}
+
+	/**
+		Returns an iterator of all child nodes which are Elements.
+		Only works if the current node is an Element or a Document.
+	**/
+	public function elements():Iterator<Xml> {
+		ensureElementType();
+		var ret = [for (child in children) if (child.nodeType == Element) child];
+		return ret.iterator();
+	}
+
+	/**
+		Returns an iterator of all child nodes which are Elements with the given nodeName.
+		Only works if the current node is an Element or a Document.
+	**/
+	public function elementsNamed(name:String):Iterator<Xml> {
+		ensureElementType();
+		var ret = [
+			for (child in children)
+				if (child.nodeType == Element && child.nodeName == name) child
+		];
+		return ret.iterator();
+	}
+
+	/**
+		Returns the first child node.
+	**/
+	public #if !cppia inline #end function firstChild():Xml {
+		ensureElementType();
+		return children[0];
+	}
+
+	/**
+		Returns the first child node which is an Element.
+	**/
+	public function firstElement():Xml {
+		ensureElementType();
+		for (child in children) {
+			if (child.nodeType == Element) {
+				return child;
+			}
+		}
+		return null;
+	}
+
+	/**
+		Adds a child node to the Document or Element.
+		A child node can only be inside one given parent node, which is indicated by the `parent` property.
+		If the child is already inside this Document or Element, it will be moved to the last position among the Document or Element's children.
+		If the child node was previously inside a different node, it will be moved to this Document or Element.
+	**/
+	public function addChild(x:Xml):Void {
+		ensureElementType();
+		if (x.parent != null) {
+			x.parent.removeChild(x);
+		}
+		children.push(x);
+		x.parent = this;
+	}
+
+	/**
+		Removes a child from the Document or Element.
+		Returns true if the child was successfuly removed.
+	**/
+	public function removeChild(x:Xml):Bool {
+		ensureElementType();
+		if (children.remove(x)) {
+			x.parent = null;
+			return true;
+		}
+		return false;
+	}
+
+	/**
+		Inserts a child at the given position among the other childs.
+		A child node can only be inside one given parent node, which is indicated by the [parent] property.
+		If the child is already inside this Document or Element, it will be moved to the new position among the Document or Element's children.
+		If the child node was previously inside a different node, it will be moved to this Document or Element.
+	**/
+	public function insertChild(x:Xml, pos:Int):Void {
+		ensureElementType();
+		if (x.parent != null) {
+			x.parent.children.remove(x);
+		}
+		children.insert(pos, x);
+		x.parent = this;
+	}
+
+	/**
+		Returns a String representation of the Xml node.
+	**/
+	public #if !cppia inline #end function toString():String {
+		return haxe.xml.Printer.print(this);
+	}
+
+	function new(nodeType:XmlType) {
+		this.nodeType = nodeType;
+		children = [];
+		attributeMap = new Map();
+	}
+
+	inline function ensureElementType() {
+		if (nodeType != Document && nodeType != Element) {
+			throw 'Bad node type, expected Element or Document but found $nodeType';
+		}
+	}
+}
\ No newline at end of file
diff --git a/source/funkin/backend/utils/CoolUtil.hx b/source/funkin/backend/utils/CoolUtil.hx
index 8cf268e6..9448b05a 100644
--- a/source/funkin/backend/utils/CoolUtil.hx
+++ b/source/funkin/backend/utils/CoolUtil.hx
@@ -718,6 +718,16 @@ class CoolUtil
 
 	@:noUsing public static inline function flxeaseFromString(mainEase:String, suffix:String)
 		return Reflect.field(FlxEase, mainEase + (mainEase == "linear" ? "" : suffix));
+
+	/*
+	 * Returns the filename of a path, without the extension.
+	 * @param path Path to get the filename from
+	 * @return Filename
+	 */
+	 @:noUsing public static inline function getFilename(file:String) {
+		var file = new haxe.io.Path(file);
+		return file.file;
+	}
 }
 
 /**
diff --git a/source/funkin/editors/character/CharacterInfoScreen.hx b/source/funkin/editors/character/CharacterInfoScreen.hx
index 28425c2e..882df52b 100644
--- a/source/funkin/editors/character/CharacterInfoScreen.hx
+++ b/source/funkin/editors/character/CharacterInfoScreen.hx
@@ -119,7 +119,7 @@ class CharacterInfoScreen extends UISubstateWindow {
 		add(isPlayerCheckbox);
 		addLabelOn(isPlayerCheckbox, "Is Player");
 
-		isGFCheckbox = new UICheckbox(isPlayerCheckbox.x + 128, positionXStepper.y + 10 + 32 + 28, "isGF", character.isGF);
+		isGFCheckbox = new UICheckbox(isPlayerCheckbox.x + 128, positionXStepper.y + 10 + 32 + 28, "isGF", false);
 		add(isGFCheckbox);
 		addLabelOn(isGFCheckbox, "Is GF");
 
@@ -168,7 +168,6 @@ class CharacterInfoScreen extends UISubstateWindow {
 
 		var xml = Xml.createElement("character");
 		xml.set("isPlayer", isPlayerCheckbox.checked ? "true" : "false");
-		xml.set("isGF", isGFCheckbox.checked ? "true" : "false");
 		xml.set("x", Std.string(positionXStepper.value));
 		xml.set("y", Std.string(positionYStepper.value));
 		xml.set("gameOverChar", gameOverCharTextBox.label.text);
diff --git a/source/funkin/game/Character.hx b/source/funkin/game/Character.hx
index 69afc69e..c2faaafc 100644
--- a/source/funkin/game/Character.hx
+++ b/source/funkin/game/Character.hx
@@ -1,37 +1,34 @@
 package funkin.game;
 
-import flixel.util.FlxColor;
-import funkin.backend.FunkinSprite;
+import flixel.util.typeLimit.OneOfTwo;
 import flixel.graphics.frames.FlxFrame;
 import flixel.math.FlxPoint;
+import flixel.math.FlxRect;
+import flixel.util.FlxColor;
+import funkin.backend.FunkinSprite;
+import funkin.backend.scripting.DummyScript;
+import funkin.backend.scripting.Script;
+import funkin.backend.scripting.events.DanceEvent;
+import funkin.backend.scripting.events.DirectionAnimEvent;
+import funkin.backend.scripting.events.PlayAnimEvent;
+import funkin.backend.scripting.events.PlayAnimEvent.PlayAnimContext;
+import funkin.backend.scripting.events.PointEvent;
+import funkin.backend.system.Conductor;
 import funkin.backend.system.interfaces.IBeatReceiver;
 import funkin.backend.system.interfaces.IOffsetCompatible;
 import funkin.backend.utils.XMLUtil;
-import flixel.math.FlxRect;
-import haxe.xml.Access;
 import haxe.Exception;
 import haxe.io.Path;
-import funkin.backend.system.Conductor;
+import haxe.xml.Access;
 import openfl.geom.ColorTransform;
-import funkin.backend.scripting.DummyScript;
-import funkin.backend.scripting.Script;
-import funkin.backend.scripting.events.*;
-import funkin.backend.scripting.events.PlayAnimEvent.PlayAnimContext;
 
 using StringTools;
 
 @:allow(funkin.desktop.editors.CharacterEditor)
 @:allow(funkin.game.StrumLine)
 @:allow(funkin.game.PlayState)
-class Character extends FunkinSprite implements IBeatReceiver implements IOffsetCompatible
-{
-	private var __stunnedTime:Float = 0;
-	private var __lockAnimThisFrame:Bool = false;
-
-	public var stunned(default, set):Bool = false;
-
+class Character extends FunkinSprite implements IBeatReceiver implements IOffsetCompatible {
 	public var isPlayer:Bool = false;
-	public var isGF:Bool = false;
 	public var curCharacter:String = 'bf';
 	public var sprite:String = 'bf';
 
@@ -42,163 +39,84 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 
 	public var icon:String = null;
 	public var iconColor:Null<FlxColor> = null;
-	public var gameOverCharacter:String = "bf-dead";
+	public var gameOverCharacter:String = Character.FALLBACK_DEAD_CHARACTER;
 
-	public var cameraOffset:FlxPoint = new FlxPoint(0, 0);
-	public var globalOffset:FlxPoint = new FlxPoint(0, 0);
+	public var cameraOffset:FlxPoint = FlxPoint.get(0, 0);
+	public var globalOffset:FlxPoint = FlxPoint.get(0, 0);
 
 	public var script:Script;
 	public var xml:Access;
 
-	public var shadowFrame:CharacterShadowFrame;
 	public var idleSuffix:String = "";
+	public var stunned(default, set):Bool = false;
 
-	public var anims = ["singLEFT", "singDOWN", "singUP", "singRIGHT"];  // Not making these inline so theyre editable through scripts!  - Nex
+	@:noCompletion var __stunnedTime:Float = 0;
+	@:noCompletion var __lockAnimThisFrame:Bool = false;
 
-	public inline function getCameraPosition()
-	{
-		var midpoint = getMidpoint();
-		var event = EventManager.get(PointEvent).recycle(
-			midpoint.x + (isPlayer ? -100 : 150) + globalOffset.x + cameraOffset.x,
-			midpoint.y - 100 + globalOffset.y + cameraOffset.y);
-		script.call("onGetCamPos", [event]);
-		midpoint.put();
-		// this event cannot be cancelled
-		return new FlxPoint(event.x, event.y);
-	}
+	@:noCompletion var __switchAnims:Bool = true;
 
-	public inline function getSingAnim(direction:Int, suffix:String = ""):String
-	{
-		return anims[direction] + suffix;
-	}
-
-	/**
-	 * Like `playSingAnimUnsafe` but checks if the character has the animation with the suffix part, otherwhise tries to play the animation without the suffix part.
-	 */
-	public function playSingAnim(direction:Int, suffix:String = "", Context:PlayAnimContext = SING, Force:Bool = true, Reversed:Bool = false, Frame:Int = 0)
-	{
-		var event = EventManager.get(DirectionAnimEvent).recycle(getSingAnim(direction, suffix), direction, suffix, Context, Reversed, Frame, Force);
-		script.call("onPlaySingAnim", [event]);
-		if (!event.cancelled)
-			playSingAnimUnsafe(event.direction, hasAnimation(event.animName) ? event.suffix : "", event.context, event.force, event.reversed, event.frame);
-	}
-
-	public function playSingAnimUnsafe(direction:Int, suffix:String = "", Context:PlayAnimContext = SING, Force:Bool = true, Reversed:Bool = false, Frame:Int = 0)
-	{
-		var event = EventManager.get(DirectionAnimEvent).recycle(getSingAnim(direction, suffix), direction, suffix, Context, Reversed, Frame, Force);
-		script.call("onPlaySingAnimUnsafe", [event]);
-		if (!event.cancelled)
-			playAnim(event.animName, event.force, event.context, event.reversed, event.frame);
-	}
-
-	public function new(x:Float, y:Float, ?character:String = "bf", isPlayer:Bool = false, switchAnims:Bool = true)
-	{
+	public function new(x:Float, y:Float, ?character:String = "bf", isPlayer:Bool = false, switchAnims:Bool = true, disableScripts:Bool = false) {
 		super(x, y);
 
 		animOffsets = new Map<String, FlxPoint>();
 		curCharacter = character;
 		this.isPlayer = isPlayer;
+		__switchAnims = switchAnims;
 
 		antialiasing = true;
 
-		while (true)
-		{
-			switch (curCharacter)
-			{
-				// case 'your-char': // To hardcode characters
-				default:
-					// load xml
-					var xmlPath = Paths.xml('characters/$curCharacter');
-					if (!Assets.exists(xmlPath))
-					{
-						curCharacter = "bf";
-						continue;
-					}
+		xml = getXMLFromCharName(this);
+
+		if(!disableScripts)
+			script = Script.create(Paths.script(Path.withoutExtension(Paths.xml('characters/$curCharacter')), null, true));
+		else
+			script = new DummyScript(curCharacter);
+		script.setParent(this);
+		script.load();
+
+		buildCharacter(xml);
+		script.call("create");
 
-					var plainXML = Assets.getText(xmlPath);
-					try
-					{
-						var charXML = Xml.parse(plainXML).firstElement();
-						if (charXML == null) throw new Exception("Missing \"character\" node in XML.");
-						xml = new Access(charXML);
-					} catch (e) {
-						Logs.trace('Error while loading character ${curCharacter}: ${e}', ERROR);
-						curCharacter = "bf";
-						continue;
-					}
-					// Loads the script
-					script = Script.create(Paths.script(Path.withoutExtension(xmlPath), null, true));
-					script.setParent(this);
-					script.load();
-					applyXML(xml);
-					script.call("create");
-			}
-			break;
-		}
 		if (script == null)
 			script = new DummyScript(curCharacter);
 
-		/**
-			NON CONVERTED CHARACTERS - DO NOT REMOVE
-		**/
-		// 	case 'bf-pixel-dead':
-		// 		frames = Paths.getSparrowAtlas('weeb/bfPixelsDEAD');
-		// 		animation.addByPrefix('singUP', "BF Dies pixel", 24, false);
-		// 		animation.addByPrefix('firstDeath', "BF Dies pixel", 24, false);
-		// 		animation.addByPrefix('deathLoop', "Retry Loop", 24, true);
-		// 		animation.addByPrefix('deathConfirm', "RETRY CONFIRM", 24, false);
-		// 		animation.play('firstDeath');
-
-		// 		addOffset('firstDeath');
-		// 		addOffset('deathLoop', -37);
-		// 		addOffset('deathConfirm', -37);
-		// 		playAnim('firstDeath');
-		// 		// pixel bullshit
-		// 		setGraphicSize(Std.int(width * 6));
-		// 		updateHitbox();
-		// 		antialiasing = false;
-		// 		flipX = true;
-
 		script.call("postCreate");
 	}
 
-	public function fixChar(switchAnims:Bool = false, autoInterval:Bool = false)
-	{
+	@:noCompletion var __swappedLeftRightAnims:Bool = false;
+	@:noCompletion var __autoInterval:Bool = false;
+
+	public function fixChar(switchAnims:Bool = false, autoInterval:Bool = false) {
 		if ((isDanceLeftDanceRight = hasAnimation("danceLeft") && hasAnimation("danceRight")) && autoInterval)
 			beatInterval = 1;
+		__autoInterval = autoInterval;
 
-		// alternative to xor operator
-		// for people who dont believe it, heres the truth table
-		// [   a   ][   b   ][ a!= b ]
-		// [ true  ][ true  ][ false ]
-		// [ true  ][ false ][ true  ]
-		// [ false ][ true  ][ true  ]
-		// [ true  ][ true  ][ false ]
-		// bros provided evidence :skull:
-		if (isPlayer != playerOffsets && switchAnims)
-		{
-			// character is flipped
-			CoolUtil.switchAnimFrames(animation.getByName('singRIGHT'), animation.getByName('singLEFT'));
-			CoolUtil.switchAnimFrames(animation.getByName('singRIGHTmiss'), animation.getByName('singLEFTmiss'));
-
-			switchOffset('singLEFT', 'singRIGHT');
-			switchOffset('singLEFTmiss', 'singRIGHTmiss');
-		}
+		// character is flipped
+		if (isPlayer != playerOffsets && switchAnims) 
+			swapLeftRightAnimations();
+		
 		frameOffset.set(getAnimOffset(getAnimName()).x, getAnimOffset(getAnimName()).y);
-		if (isPlayer)
-			flipX = !flipX;
+		if (isPlayer) flipX = !flipX;
 		__baseFlipped = flipX;
 	}
 
-	var __baseFlipped:Bool = false;
-	var isDanceLeftDanceRight:Bool = false;
+	public function swapLeftRightAnimations() {
+		CoolUtil.switchAnimFrames(animation.getByName('singRIGHT'), animation.getByName('singLEFT'));
+		CoolUtil.switchAnimFrames(animation.getByName('singRIGHTmiss'), animation.getByName('singLEFTmiss'));
 
-	override function update(elapsed:Float)
-	{
+		switchOffset('singLEFT', 'singRIGHT');
+		switchOffset('singLEFTmiss', 'singRIGHTmiss');
+
+		__swappedLeftRightAnims = true;
+	}
+
+	@:noCompletion var __baseFlipped:Bool = false;
+	@:noCompletion var isDanceLeftDanceRight:Bool = false;
+
+	override function update(elapsed:Float) {
 		super.update(elapsed);
 		script.call("update", [elapsed]);
-		if (stunned)
-		{
+		if (stunned) {
 			__stunnedTime += elapsed;
 			if (__stunnedTime > 5 / 60)
 				stunned = false;
@@ -212,33 +130,20 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 
 	private var danced:Bool = false;
 
-	/**
-	 * FOR GF DANCING SHIT
-	 */
-	public function dance()
-	{
+	public function dance() {
 		if(debugMode) return;
 
 		var event = EventManager.get(DanceEvent).recycle(danced);
 		script.call("onDance", [event]);
-		if (event.cancelled)
-			return;
+		if (event.cancelled) return;
 
-		switch (curCharacter)
-		{
-			// hardcode custom dance animations here
-			default:
-				if (isDanceLeftDanceRight)
-				{
-					playAnim(((danced = !danced) ? 'danceLeft' : 'danceRight') + idleSuffix, DANCE);
-				}
-				else
-					playAnim('idle' + idleSuffix, DANCE);
-		}
+		if (isDanceLeftDanceRight)
+			playAnim(((danced = !danced) ? 'danceLeft' : 'danceRight') + idleSuffix, DANCE);
+		else
+			playAnim('idle' + idleSuffix, DANCE);
 	}
 
-	public function tryDance()
-	{
+	public function tryDance() {
 		switch (lastAnimContext) {
 			case SING | MISS:
 				if (lastHit + (Conductor.stepCrochet * holdTime) < Conductor.songPosition)
@@ -258,96 +163,61 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 	 * Whenever the character should dance on beat or not. Set to false for `gf`, since the dance animation is automatically handled by PlayState.
 	 */
 	public var danceOnBeat:Bool = true;
-
 	public override function beatHit(curBeat:Int) {
 		script.call("beatHit", [curBeat]);
 
 		if (danceOnBeat && (curBeat + beatOffset) % beatInterval == 0 && !__lockAnimThisFrame)
-		{
 			tryDance();
-		}
 	}
 
-	public override function stepHit(curStep:Int) {
+	public override function stepHit(curStep:Int)
 		script.call("stepHit", [curStep]);
-		// nothing
-	}
-
-	var __reverseDrawProcedure:Bool = false;
 
+	@:noCompletion var __reverseDrawProcedure:Bool = false;
 	public override function getScreenBounds(?newRect:FlxRect, ?camera:FlxCamera):FlxRect {
-		if (__reverseDrawProcedure)
-		{
+		if (__reverseDrawProcedure) {
 			scale.x *= -1;
-			var bounds = super.getScreenBounds(newRect, camera);
+			var bounds:FlxRect = super.getScreenBounds(newRect, camera);
 			scale.x *= -1;
 			return bounds;
 		}
 		return super.getScreenBounds(newRect, camera);
 	}
 
-	public function isFlippedOffsets() {
-		return (isPlayer != playerOffsets) != (flipX != __baseFlipped);
+	public override function isOnScreen(?camera:FlxCamera):Bool {
+		if (debugMode) return true;
+		return super.isOnScreen(camera);
 	}
 
-	var __drawingShadowFrame = false;
-	var __oldColorTransform = new ColorTransform();
+	public function isFlippedOffsets()
+		return (isPlayer != playerOffsets) != (flipX != __baseFlipped);
 
 	public override function draw() {
-		if (!__drawingShadowFrame && shadowFrame != null) {
-			__drawingShadowFrame = true;
-
-			var oldFrame = _frame;
-			var oldPos = FlxPoint.get(frameOffset.x, frameOffset.y);
-
-			__oldColorTransform.copyColorTransform(colorTransform);
-
-			colorTransform.alphaMultiplier = 1;
-			colorTransform.alphaOffset = 0;
-			colorTransform.blueMultiplier = 0;
-			colorTransform.blueOffset = 25;
-			colorTransform.greenMultiplier = 0;
-			colorTransform.greenOffset = 25;
-			colorTransform.redMultiplier = 0;
-			colorTransform.redOffset = 25;
-
-			_frame = shadowFrame.frame;
-			var o = getAnimOffset(shadowFrame.anim);
-			frameOffset.set(o.x, o.y);
-			super.draw();
-
-			_frame = oldFrame;
-			frameOffset.set(oldPos.x, oldPos.y);
-
-			colorTransform.copyColorTransform(__oldColorTransform);
-
-			oldPos.put();
-
-			__drawingShadowFrame = false;
-		}
-
-		if (isFlippedOffsets())
-		{
+		if (isFlippedOffsets()) {
 			__reverseDrawProcedure = true;
-
 			flipX = !flipX;
 			scale.x *= -1;
+
 			super.draw();
+
 			flipX = !flipX;
 			scale.x *= -1;
-
 			__reverseDrawProcedure = false;
-		}
-		else
-			super.draw();
+		} else super.draw();
 	}
 
-	public override function playAnim(AnimName:String, Force:Bool = false, Context:PlayAnimContext = NONE, Reversed:Bool = false, Frame:Int = 0):Void
-	{
+	public var singAnims = ["singLEFT", "singDOWN", "singUP", "singRIGHT"];
+	public function playSingAnim(direction:Int, suffix:String = "", Context:PlayAnimContext = SING, Force:Bool = true, Reversed:Bool = false, Frame:Int = 0) {
+		var event = EventManager.get(DirectionAnimEvent).recycle(singAnims[direction % singAnims.length] + suffix, direction, suffix, Context, Reversed, Frame, Force);
+		script.call("onPlaySingAnim", [event]);
+		if (!event.cancelled)
+			playAnim(event.animName, event.force, event.context, event.reversed, event.frame);
+	}
+
+	public override function playAnim(AnimName:String, Force:Bool = false, Context:PlayAnimContext = NONE, Reversed:Bool = false, Frame:Int = 0) {
 		var event = EventManager.get(PlayAnimEvent).recycle(AnimName, Force, Reversed, Frame, Context);
 		script.call("onPlayAnim", [event]);
-		if (event.cancelled)
-			return;
+		if (event.cancelled) return;
 
 		super.playAnim(event.animName, event.force, event.context, event.reverse, event.startingFrame);
 
@@ -356,50 +226,63 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 			lastHit = Conductor.songPosition;
 	}
 
-	public override function destroy()
-	{
+	public inline function getCameraPosition() {
+		var midpoint:FlxPoint = getMidpoint();
+		var event = EventManager.get(PointEvent).recycle(
+			midpoint.x + (isPlayer ? -100 : 150) + globalOffset.x + cameraOffset.x,
+			midpoint.y - 100 + globalOffset.y + cameraOffset.y);
+		script.call("onGetCamPos", [event]);
+
+		midpoint.put();
+		return new FlxPoint(event.x, event.y);
+	}
+
+	public override function destroy() {
 		super.destroy();
 
 		cameraOffset.put();
 		globalOffset.put();
 	}
 
-	// FlxTrail Stuff for fixing position
+	@:noCompletion var __reverseTrailProcedure:Bool = false;
 
-	var __reverseTrailProcedure:Bool = false;
-
-	/**
-	 * When using trails on characters you should do `trail.beforeCache = char.beforeTrailCache;`
-	 **/
-	dynamic function beforeTrailCache() {
-		if (isFlippedOffsets())
-		{
+	// When using trails on characters you should do `trail.beforeCache = char.beforeTrailCache;`
+	public dynamic function beforeTrailCache()
+		if (isFlippedOffsets()) {
 			flipX = !flipX;
 			scale.x *= -1;
 			__reverseTrailProcedure = true;
 		}
-	}
 
-	/**
-	 * When using trails on characters you should do `trail.afterCache = char.afterTrailCache;`
-	 **/
-	dynamic function afterTrailCache() {
-		if (__reverseTrailProcedure)
-		{
+	// When using trails on characters you should do `trail.afterCache = char.afterTrailCache;`
+	public dynamic function afterTrailCache()
+		if (__reverseTrailProcedure) {
 			flipX = !flipX;
 			scale.x *= -1;
 			__reverseTrailProcedure = false;
 		}
+
+	public function applyXML(xml:Access) { // just for now till i remake the dumb editor
+		gameOverCharacter = Character.FALLBACK_DEAD_CHARACTER;
+		cameraOffset.set(0, 0);
+		globalOffset.set(0, 0);
+		playerOffsets = false;
+		flipX = false;
+		holdTime = 4;
+		iconColor = null;
+
+		animation.destroyAnimations();
+		animDatas.clear();
+
+		__baseFlipped = false;
+		buildCharacter(xml);
 	}
 
-	// Character editor and loading
-
-	public function applyXML(xml:Access) {
-		this.xml = xml; // modders wassup
+	public inline function buildCharacter(xml:Access) {
+		this.xml = xml; // Modders wassup :D
 		sprite = curCharacter;
 
 		if (xml.x.exists("isPlayer")) playerOffsets = (xml.x.get("isPlayer") == "true");
-		if (xml.x.exists("isGF")) isGF = (xml.x.get("isGF") == "true");
 		if (xml.x.exists("x")) globalOffset.x = Std.parseFloat(xml.x.get("x"));
 		if (xml.x.exists("y")) globalOffset.y = Std.parseFloat(xml.x.get("y"));
 		if (xml.x.exists("gameOverChar")) gameOverCharacter = xml.x.get("gameOverChar");
@@ -410,7 +293,7 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 		if (xml.x.exists("icon")) icon = xml.x.get("icon");
 		if (xml.x.exists("color")) iconColor = FlxColor.fromString(xml.x.get("color"));
 		if (xml.x.exists("scale")) {
-			var scale = Std.parseFloat(xml.x.get("scale")).getDefault(1);
+			var scale:Float = Std.parseFloat(xml.x.get("scale")).getDefault(1);
 			this.scale.set(scale, scale);
 			updateHitbox();
 		}
@@ -420,71 +303,85 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 		var hasInterval:Bool = xml.x.exists("interval");
 		if (hasInterval) beatInterval = Std.parseInt(xml.x.get("interval"));
 
-		var tempList = ["isPlayer", "isGF", "x", "y", "gameOverChar", "camx", "camy", "holdTime", "flipX", "icon", "color", "scale", "antialiasing", "sprite", "interval"];
-		var atts = [for (i in xml.x.attributes()) i];
-		for (i in atts)
-			if (!tempList.contains(i)) {
-				extra[i] = xml.x.get(i);
-			}
-
 		loadSprite(Paths.image('characters/$sprite'));
 
-		animation.destroyAnimations();
-		animDatas.clear();
 		for (anim in xml.nodes.anim)
-		{
 			XMLUtil.addXMLAnimation(this, anim);
-		}
 
-		fixChar(true, !hasInterval);
+		for (attribute in xml.x.attributes())
+			if (!characterProperties.contains(attribute))
+				extra[attribute] = xml.x.get(attribute);
+
+		fixChar(__switchAnims, !hasInterval);
 		dance();
 	}
 
-	public function buildXML(?animsOrder:Array<String>):Xml {
-		var xml = Xml.createElement("character");
-		xml.set("isPlayer", playerOffsets == true ? "true" : "false");
-		xml.set("isGF", isGF == true ? "true" : "false");
-		xml.set("x", Std.string(globalOffset.x));
-		xml.set("y", Std.string(globalOffset.y));
-		xml.set("gameOverChar", gameOverCharacter);
-		xml.set("camx", Std.string(cameraOffset.x));
-		xml.set("camy", Std.string(cameraOffset.y));
-		xml.set("holdTime", Std.string(holdTime));
-		xml.set("flipX", Std.string(flipX));
-		xml.set("icon", getIcon());
-		if (iconColor != null)
-			xml.set("color", iconColor.toWebString());
-		xml.set("scale", Std.string(scale.x));
-		xml.set("antialiasing", antialiasing == true ? "true" : "false");
-		xml.set("sprite", sprite);
+	public static var characterProperties:Array<String> = [
+		"x", "y", "sprite", "scale", "antialiasing", 
+		"flipX", "camx", "camy", "isPlayer", "icon", 
+		"color", "gameOverChar", "holdTime"
+	];
+	public static var characterAnimProperties:Array<String> = [
+		"name", "anim", "x", "y", "fps", "loop", "indices"
+	];
 
-		for (prop=>val in extra) {
-			xml.set(prop, Std.string(val));
-		}
+	public inline function buildXML(?animsOrder:Array<String>):Xml {
+		var xml:Xml = Xml.createElement("character");
+		xml.attributeOrder = characterProperties.copy();
+		
+		if (globalOffset.x != 0) xml.set("x", Std.string(FlxMath.roundDecimal(globalOffset.x, 2)));
+		if (globalOffset.y != 0) xml.set("y", Std.string(FlxMath.roundDecimal(globalOffset.y, 2)));
+
+		if (cameraOffset.x != 0) xml.set("camx", Std.string(FlxMath.roundDecimal(cameraOffset.x, 2)));
+		if (cameraOffset.y != 0) xml.set("camy", Std.string(FlxMath.roundDecimal(cameraOffset.y, 2)));
+
+		if (holdTime != 4) xml.set("holdTime", Std.string(FlxMath.roundDecimal(holdTime, 4)));
+
+		if (flipX) xml.set("flipX", Std.string(flipX));
+		xml.set("icon", getIcon());
+
+		if (gameOverCharacter != Character.FALLBACK_DEAD_CHARACTER) xml.set("gameOverChar", gameOverCharacter);
+		if (iconColor != null) xml.set("color", iconColor.toWebString());
+
+		xml.set("sprite", sprite);
+		if (scale.x != 1) xml.set("scale", Std.string(FlxMath.roundDecimal(scale.x, 4)));
+		if (!antialiasing) xml.set("antialiasing", antialiasing == true ? "true" : "false");
+
+		if (playerOffsets) xml.set("isPlayer", playerOffsets == true ? "true" : "false");
 
 		var anims:Array<AnimData> = [];
 		if (animsOrder != null) {
 			for (name in animsOrder)
 				if (animDatas.exists(name)) anims.push(animDatas.get(name));
 		} else
-			anims = [for (anim in animDatas) anim];
+			anims = Lambda.array(animDatas);
 
-		for (anim in anims)
-		{
+		for (anim in anims) {
 			var animXml:Xml = Xml.createElement('anim');
+			animXml.attributeOrder = characterAnimProperties;
+
 			animXml.set("name", anim.name);
 			animXml.set("anim", anim.anim);
 			animXml.set("loop", Std.string(anim.loop));
-			animXml.set("fps", Std.string(anim.fps));
+			animXml.set("fps", Std.string(FlxMath.roundDecimal(anim.fps, 2)));
+
 			var offset:FlxPoint = getAnimOffset(anim.name);
-			animXml.set("x", Std.string(offset.x));
-			animXml.set("y", Std.string(offset.y));
+			animXml.set("x", Std.string(FlxMath.roundDecimal(offset.x, 2)));
+			animXml.set("y", Std.string(FlxMath.roundDecimal(offset.y, 2)));
 			offset.putWeak();
+
 			if (anim.indices.length > 0)
 				animXml.set("indices", anim.indices.join(","));
+
 			xml.addChild(animXml);
 		}
 
+		for (name => val in extra)
+			if (!xml.attributeOrder.contains(name)) {
+				xml.attributeOrder.push(name);
+				xml.set(name, Std.string(val));
+			}
+
 		this.xml = new Access(xml);
 		return xml;
 	}
@@ -492,74 +389,72 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 	public inline function getIcon()
 		return (icon != null) ? icon : curCharacter;
 
-	public function getAnimOrder() {
+	public function getAnimOrder()
 		return [for(a in xml.nodes.anim) if(a.has.name) a.att.name];
-	}
 
-	// Getters / Setters
-
-	@:noCompletion private function set_stunned(b:Bool)
-	{
+	@:noCompletion private function set_stunned(b:Bool) {
 		__stunnedTime = 0;
 		return stunned = b;
 	}
 
-	// Backwards Compat
-
-	/**
-	 * Interval at which the character will dance (higher number = slower dance)
-	 */
+	// Interval at which the character will dance (higher number = slower dance)
 	@:noCompletion public var danceInterval(get, set):Int;
 	@:noCompletion private function set_danceInterval(v:Int)
 		return beatInterval = v;
 	@:noCompletion private function get_danceInterval()
 		return beatInterval;
 
-	// Statics
 
-	public static function getIconFromCharName(?curCharacter:String) {
-		if(curCharacter == null) return "face";
-		var icon = curCharacter;
-		while(true) {
-			switch (curCharacter) {
-				// case 'your-char': // To hardcode characters icons
-				default:
-					// load xml
-					var xmlPath = Paths.xml('characters/$curCharacter');
-					if (!Assets.exists(xmlPath)) {
-						curCharacter = "bf";
-						continue;
-					}
+	public static var FALLBACK_CHARACTER:String = "bf";
+	public static var FALLBACK_DEAD_CHARACTER:String = "bf-dead";
+	public static function getXMLFromCharName(character:OneOfTwo<String, Character>):Access {
+		var char:Character = null;
+		if (character is Character) {
+			char = cast(character, Character);
+			character = char.curCharacter;
+		}
 
-					var xml = null;
-					var plainXML = Assets.getText(xmlPath);
-					try {
-						var charXML = Xml.parse(plainXML).firstElement();
-						if (charXML == null) throw new Exception("Missing \"character\" node in XML.");
-						xml = new Access(charXML);
-					} catch(e) {
-						Logs.trace('Error while loading character ${curCharacter}: ${e}', ERROR);
-						curCharacter = "bf";
-						continue;
-					}
+		var xml:Access = null;
+		while (true) {
+			var xmlPath:String = Paths.xml('characters/$character');
+			if (!Assets.exists(xmlPath)) {
+				character = FALLBACK_CHARACTER;
+				if (char != null) 
+					char.curCharacter = character;
+				continue;
+			}
 
-					if (xml.x.exists("icon")) icon = xml.x.get("icon");
-				}
+			var plainXML:String = Assets.getText(xmlPath);
+			try {
+				var charXML:Xml = Xml.parse(plainXML).firstElement();
+				if (charXML == null) throw new Exception("Missing \"character\" node in XML.");
+				xml = new Access(charXML);
+			} catch (e) {
+				Logs.trace('Error while loading character ${character}: ${e}', ERROR);
+
+				character = FALLBACK_CHARACTER;
+				if (char != null) 
+					char.curCharacter = character;
+				continue;
+			}
 			break;
 		}
+		return xml;
+	}
+
+	public static function getIconFromCharName(?character:String) {
+		if(character == null) return "face";
+		var icon:String = character;
+
+		var xml:Access = getXMLFromCharName(character);
+		if (xml != null && xml.x.exists("icon")) icon = xml.x.get("icon");
+
 		return icon;
 	}
 
-	public static function getList(?mods:Bool = false):Array<String> {
+	public static function getList(?mods:Bool = false):Array<String>
 		return [
 			for (path in Paths.getFolderContent('data/characters/', true, mods ? MODS : BOTH))
-				if (Path.extension(path) == "xml") Path.withoutDirectory(Path.withoutExtension(path))
+				if (Path.extension(path) == "xml") CoolUtil.getFilename(path)
 		];
-	}
-}
-
-typedef CharacterShadowFrame =
-{
-	var anim:String;
-	var frame:FlxFrame;
-}
+}
\ No newline at end of file
diff --git a/source/haxe/xml/Printer.hx b/source/haxe/xml/Printer.hx
new file mode 100644
index 00000000..ab2e4cde
--- /dev/null
+++ b/source/haxe/xml/Printer.hx
@@ -0,0 +1,137 @@
+/*
+* Copyright (C)2005-2019 Haxe Foundation
+*
+* Permission is hereby granted, free of charge, to any person obtaining a
+* copy of this software and associated documentation files (the "Software"),
+* to deal in the Software without restriction, including without limitation
+* the rights to use, copy, modify, merge, publish, distribute, sublicense,
+* and/or sell copies of the Software, and to permit persons to whom the
+* Software is furnished to do so, subject to the following conditions:
+*
+* The above copyright notice and this permission notice shall be included in
+* all copies or substantial portions of the Software.
+*
+* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+* DEALINGS IN THE SOFTWARE.
+*/
+
+package haxe.xml;
+
+using StringTools;
+
+/**
+ This class provides utility methods to convert Xml instances to
+	String representation.
+**/
+class Printer {
+	/**
+	 Convert `Xml` to string representation.
+
+		Set `pretty` to `true` to prettify the result.
+	**/
+	static public function print(xml:Xml, ?pretty = false) {
+		var printer = new Printer(pretty);
+		printer.writeNode(xml, "");
+		return printer.output.toString();
+	}
+
+	var output:StringBuf;
+	var pretty:Bool;
+
+	function new(pretty) {
+		output = new StringBuf();
+		this.pretty = pretty;
+	}
+
+	function writeNode(value:Xml, tabs:String) {
+		switch (value.nodeType) {
+			case CData:
+				write(tabs + "<![CDATA[");
+				write(value.nodeValue);
+				write("]]>");
+				newline();
+			case Comment:
+				var commentContent:String = value.nodeValue;
+				commentContent = ~/[\n\r\t]+/g.replace(commentContent, "");
+				commentContent = "<!--" + commentContent + "-->";
+				write(tabs);
+				write(StringTools.trim(commentContent));
+				newline();
+			case Document:
+				for (child in value) {
+					writeNode(child, tabs);
+				}
+			case Element:
+				write(tabs + "<");
+				write(value.nodeName);
+
+				var attributes:Array<String> = value.attributeOrder;
+				if (attributes == null)
+					attributes = [for (att in value.attributes()) att];
+
+				for (attribute in attributes) {
+					if (!value.exists(attribute) || value.get(attribute).trim() == "") continue;
+
+					write(" " + attribute + "=\"");
+					write(StringTools.htmlEscape(value.get(attribute), true));
+					write("\"");
+				}
+				if (hasChildren(value)) {
+					write(">");
+					newline();
+					for (child in value) {
+						writeNode(child, pretty ? tabs + "\t" : tabs);
+					}
+					write(tabs + "</");
+					write(value.nodeName);
+					write(">");
+					newline();
+				} else {
+					write("/>");
+					newline();
+				}
+			case PCData:
+				var nodeValue:String = value.nodeValue;
+				if (nodeValue.length != 0) {
+					write(tabs + StringTools.htmlEscape(nodeValue));
+					newline();
+				}
+			case ProcessingInstruction:
+				write("<?" + value.nodeValue + "?>");
+				newline();
+			case DocType:
+				write("<!DOCTYPE " + value.nodeValue + ">");
+				newline();
+		}
+	}
+
+	inline function write(input:String) {
+		output.add(input);
+	}
+
+	inline function newline() {
+		if (pretty) {
+			output.add("\n");
+		}
+	}
+
+	function hasChildren(value:Xml):Bool {
+		for (child in value) {
+			switch (child.nodeType) {
+				case Element, PCData:
+					return true;
+				case CData, Comment:
+					if (StringTools.ltrim(child.nodeValue).length != 0) {
+						return true;
+					}
+				case _:
+			}
+		}
+		return false;
+	}
+}
\ No newline at end of file

From 823c1a7f6b57cbe9642cb4f4ebe2c7a9e73b45ac Mon Sep 17 00:00:00 2001
From: lunarcleint <88809950+lunarcleint@users.noreply.github.com>
Date: Wed, 12 Jun 2024 19:11:42 -0700
Subject: [PATCH 02/12] PLEASE WORK PLEASE WORK PLEASEEEE

---
 source/funkin/editors/character/CharacterGhostsHandler.hx | 2 +-
 source/funkin/game/Character.hx                           | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/source/funkin/editors/character/CharacterGhostsHandler.hx b/source/funkin/editors/character/CharacterGhostsHandler.hx
index 62a44c9f..2d8364fb 100644
--- a/source/funkin/editors/character/CharacterGhostsHandler.hx
+++ b/source/funkin/editors/character/CharacterGhostsHandler.hx
@@ -17,7 +17,7 @@ class CharacterGhostsHandler extends FlxTypedGroup<Character> {
 	}
 
 	public function createGhost(anim:String) {
-		var ghost:Character = new Character(0,0, character.curCharacter);
+		var ghost:Character = new Character(0,0, character.curCharacter, false, false);
 		ghost.applyXML(character.xml); // apply cur character stuff
 		ghost.playAnim(anim);
 		ghost.stopAnimation();
diff --git a/source/funkin/game/Character.hx b/source/funkin/game/Character.hx
index c2faaafc..df16d7c6 100644
--- a/source/funkin/game/Character.hx
+++ b/source/funkin/game/Character.hx
@@ -108,6 +108,7 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 		switchOffset('singLEFTmiss', 'singRIGHTmiss');
 
 		__swappedLeftRightAnims = true;
+		trace("fuck");
 	}
 
 	@:noCompletion var __baseFlipped:Bool = false;
@@ -190,7 +191,7 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 	}
 
 	public function isFlippedOffsets()
-		return (isPlayer != playerOffsets) != (flipX != __baseFlipped);
+		return __switchAnims && ((isPlayer != playerOffsets) != (flipX != __baseFlipped));
 
 	public override function draw() {
 		if (isFlippedOffsets()) {
@@ -312,6 +313,7 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 			if (!characterProperties.contains(attribute))
 				extra[attribute] = xml.x.get(attribute);
 
+		trace(__switchAnims);
 		fixChar(__switchAnims, !hasInterval);
 		dance();
 	}

From 7b5d975072690071824d21034485bf47c4396e8c Mon Sep 17 00:00:00 2001
From: ~ Care <126361810+usb-port-2@users.noreply.github.com>
Date: Mon, 17 Jun 2024 18:35:11 +0100
Subject: [PATCH 03/12] Icon bop toggle (#320)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Icon bop toggle

* Disabled Icon lerping too

* fixin the format n shit

---------

Co-authored-by: ⍚~Nex <87421482+NexIsDumb@users.noreply.github.com>
---
 source/funkin/game/PlayState.hx | 28 ++++++++++++++++++----------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/source/funkin/game/PlayState.hx b/source/funkin/game/PlayState.hx
index f585e89f..039d9d33 100644
--- a/source/funkin/game/PlayState.hx
+++ b/source/funkin/game/PlayState.hx
@@ -211,9 +211,13 @@ class PlayState extends MusicBeatState
 	 */
 	public var canAccessDebugMenus:Bool = true;
 	/**
-	 * Wether or not to show the secret gitaroo pause.
+	 * Whether or not to show the secret gitaroo pause.
 	 */
 	public var allowGitaroo:Bool = true;
+	/**
+	 * Whether or not to bop the icons on beat.
+	 */
+	public var doIconBop:Bool = true;
 
 	/**
 	 * Whenever cam zooming is enabled, enables on a note hit if not cancelled.
@@ -1231,12 +1235,13 @@ class PlayState extends MusicBeatState
 			}
 		}
 
-		iconP1.scale.set(lerp(iconP1.scale.x, 1, 0.33), lerp(iconP1.scale.y, 1, 0.33));
-		iconP2.scale.set(lerp(iconP2.scale.x, 1, 0.33), lerp(iconP2.scale.y, 1, 0.33));
-
-		iconP1.updateHitbox();
-		iconP2.updateHitbox();
+                if (doIconBop) {
+			iconP1.scale.set(lerp(iconP1.scale.x, 1, 0.33), lerp(iconP1.scale.y, 1, 0.33));
+			iconP2.scale.set(lerp(iconP2.scale.x, 1, 0.33), lerp(iconP2.scale.y, 1, 0.33));
 
+			iconP1.updateHitbox();
+			iconP2.updateHitbox();
+		}
 		updateIconPositions();
 
 		if (startingSong)
@@ -1784,11 +1789,14 @@ class PlayState extends MusicBeatState
 			camHUD.zoom += 0.03 * camZoomingStrength;
 		}
 
-		iconP1.scale.set(1.2, 1.2);
-		iconP2.scale.set(1.2, 1.2);
+                if (doIconBop)
+		{
+			iconP1.scale.set(1.2, 1.2);
+			iconP2.scale.set(1.2, 1.2);
 
-		iconP1.updateHitbox();
-		iconP2.updateHitbox();
+			iconP1.updateHitbox();
+			iconP2.updateHitbox();
+		}
 
 		scripts.call("beatHit", [curBeat]);
 	}

From 2afc3bf894c18912c809045942384d3e15a711b8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=8D=9A=7ENex?=
 <87421482+NexIsDumb@users.noreply.github.com>
Date: Mon, 17 Jun 2024 19:38:22 +0200
Subject: [PATCH 04/12] go fuck yourself github

---
 source/funkin/game/PlayState.hx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/source/funkin/game/PlayState.hx b/source/funkin/game/PlayState.hx
index 039d9d33..67d5d822 100644
--- a/source/funkin/game/PlayState.hx
+++ b/source/funkin/game/PlayState.hx
@@ -1235,7 +1235,7 @@ class PlayState extends MusicBeatState
 			}
 		}
 
-                if (doIconBop) {
+        if (doIconBop) {
 			iconP1.scale.set(lerp(iconP1.scale.x, 1, 0.33), lerp(iconP1.scale.y, 1, 0.33));
 			iconP2.scale.set(lerp(iconP2.scale.x, 1, 0.33), lerp(iconP2.scale.y, 1, 0.33));
 
@@ -1789,7 +1789,7 @@ class PlayState extends MusicBeatState
 			camHUD.zoom += 0.03 * camZoomingStrength;
 		}
 
-                if (doIconBop)
+        if (doIconBop)
 		{
 			iconP1.scale.set(1.2, 1.2);
 			iconP2.scale.set(1.2, 1.2);

From 576448a165122c7b7bf90eb678cdf1f5076f9c4f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=8D=9A=7ENex?=
 <87421482+NexIsDumb@users.noreply.github.com>
Date: Mon, 17 Jun 2024 22:49:38 +0200
Subject: [PATCH 05/12] Revert "PLEASE WORK PLEASE WORK PLEASEEEE"

This reverts commit 823c1a7f6b57cbe9642cb4f4ebe2c7a9e73b45ac.
---
 source/funkin/editors/character/CharacterGhostsHandler.hx | 2 +-
 source/funkin/game/Character.hx                           | 4 +---
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/source/funkin/editors/character/CharacterGhostsHandler.hx b/source/funkin/editors/character/CharacterGhostsHandler.hx
index 2d8364fb..62a44c9f 100644
--- a/source/funkin/editors/character/CharacterGhostsHandler.hx
+++ b/source/funkin/editors/character/CharacterGhostsHandler.hx
@@ -17,7 +17,7 @@ class CharacterGhostsHandler extends FlxTypedGroup<Character> {
 	}
 
 	public function createGhost(anim:String) {
-		var ghost:Character = new Character(0,0, character.curCharacter, false, false);
+		var ghost:Character = new Character(0,0, character.curCharacter);
 		ghost.applyXML(character.xml); // apply cur character stuff
 		ghost.playAnim(anim);
 		ghost.stopAnimation();
diff --git a/source/funkin/game/Character.hx b/source/funkin/game/Character.hx
index df16d7c6..c2faaafc 100644
--- a/source/funkin/game/Character.hx
+++ b/source/funkin/game/Character.hx
@@ -108,7 +108,6 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 		switchOffset('singLEFTmiss', 'singRIGHTmiss');
 
 		__swappedLeftRightAnims = true;
-		trace("fuck");
 	}
 
 	@:noCompletion var __baseFlipped:Bool = false;
@@ -191,7 +190,7 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 	}
 
 	public function isFlippedOffsets()
-		return __switchAnims && ((isPlayer != playerOffsets) != (flipX != __baseFlipped));
+		return (isPlayer != playerOffsets) != (flipX != __baseFlipped);
 
 	public override function draw() {
 		if (isFlippedOffsets()) {
@@ -313,7 +312,6 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 			if (!characterProperties.contains(attribute))
 				extra[attribute] = xml.x.get(attribute);
 
-		trace(__switchAnims);
 		fixChar(__switchAnims, !hasInterval);
 		dance();
 	}

From a6c5b4d6e81d1c5d2012388e69a906b7287599b3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=8D=9A=7ENex?=
 <87421482+NexIsDumb@users.noreply.github.com>
Date: Tue, 18 Jun 2024 00:04:26 +0200
Subject: [PATCH 06/12] fixin some stuff about file attribs

Co-Authored-By: Ne_Eo <NeeEoo@users.noreply.github.com>
---
 source/funkin/backend/utils/CoolUtil.hx       |  35 ++-
 .../backend/utils/FileAttributeWrapper.hx     | 292 ++++++++++++++++++
 source/funkin/backend/utils/NativeAPI.hx      |  30 +-
 3 files changed, 354 insertions(+), 3 deletions(-)
 create mode 100644 source/funkin/backend/utils/FileAttributeWrapper.hx

diff --git a/source/funkin/backend/utils/CoolUtil.hx b/source/funkin/backend/utils/CoolUtil.hx
index 9448b05a..979662e0 100644
--- a/source/funkin/backend/utils/CoolUtil.hx
+++ b/source/funkin/backend/utils/CoolUtil.hx
@@ -100,7 +100,7 @@ class CoolUtil
 
 	/**
 	 * Sets an attribute to a file or a folder adding eventual missing folders in the path
-	 * (WARNING: Only works on `windows`. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
+	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
 	 * @param path Path to the file or folder
 	 * @param attrib The attribute to set (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
 	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
@@ -114,6 +114,39 @@ class CoolUtil
 		return result;
 	}
 
+	/**
+	 * Sets file attributes to a file or a folder adding eventual missing folders in the path
+	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
+	 * @param path Path to the file or folder
+	 * @param attrib The attribute to set (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
+	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
+	 * @return The result code: 0 means that it failed setting
+	 */
+	 @:noUsing public static inline function safeSetAttributes(path:String, attrib:OneOfTwo<FileAttributeWrapper, Int>, useAbsol:Bool = true) {
+		addMissingFolders(Path.directory(path));
+
+		var result = NativeAPI.setFileAttributes(path, attrib, useAbsol);
+		if(result == 0) Logs.trace('Failed to set attributes to $path with a code of: $result', WARNING);
+		return result;
+	}
+
+	/**
+	 * Gets the attributes of a file or a folder adding eventual missing folders in the path
+	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
+	 * @param path Path to the file or folder
+	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
+	 * @return The result code: 0 means that it failed setting
+	 */
+	 // uncomment if it crashes when using it normall
+	/*@:noUsing public static inline function safeGetAttribute(path:String, useAbsol:Bool = true) {
+		addMissingFolders(Path.directory(path));
+
+		var result = NativeAPI.getFileAttribute(path, useAbsol);
+		if(result == 0) Logs.trace('Failed to get attribute to $path with a code of: $result', WARNING);
+		return result;
+	}*/
+
+
 	/**
 	 * Creates eventual missing folders to the specified `path`
 	 *
diff --git a/source/funkin/backend/utils/FileAttributeWrapper.hx b/source/funkin/backend/utils/FileAttributeWrapper.hx
new file mode 100644
index 00000000..0b526223
--- /dev/null
+++ b/source/funkin/backend/utils/FileAttributeWrapper.hx
@@ -0,0 +1,292 @@
+package funkin.backend.utils;
+
+import funkin.backend.utils.NativeAPI.FileAttribute;
+
+/**
+ * Currently only for Windows, but planned to work on other platforms later.
+ */
+class FileAttributeWrapper
+{
+	public var flags:Int;
+
+	public function new(flags:Int)
+	{
+		this.flags = flags;
+	}
+
+	public var isArchived(get, set):Bool;
+
+	private function get_isArchived():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.ARCHIVE) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isArchived(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.ARCHIVE;
+		else
+			flags &= ~FileAttribute.ARCHIVE;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isHidden(get, set):Bool;
+
+	private function get_isHidden():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.HIDDEN) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isHidden(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.HIDDEN;
+		else
+			flags &= ~FileAttribute.HIDDEN;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isNormal(get, set):Bool;
+
+	private function get_isNormal():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.NORMAL) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isNormal(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.NORMAL;
+		else
+			flags &= ~FileAttribute.NORMAL;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isNotContentIndexed(get, set):Bool;
+
+	private function get_isNotContentIndexed():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.NOT_CONTENT_INDEXED) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isNotContentIndexed(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.NOT_CONTENT_INDEXED;
+		else
+			flags &= ~FileAttribute.NOT_CONTENT_INDEXED;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isOffline(get, set):Bool;
+
+	private function get_isOffline():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.OFFLINE) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isOffline(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.OFFLINE;
+		else
+			flags &= ~FileAttribute.OFFLINE;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isReadOnly(get, set):Bool;
+
+	private function get_isReadOnly():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.READONLY) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isReadOnly(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.READONLY;
+		else
+			flags &= ~FileAttribute.READONLY;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isSystem(get, set):Bool;
+
+	private function get_isSystem():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.SYSTEM) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isSystem(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.SYSTEM;
+		else
+			flags &= ~FileAttribute.SYSTEM;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	public var isTemporary(get, set):Bool;
+
+	private function get_isTemporary():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.TEMPORARY) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	private function set_isTemporary(value:Bool):Bool
+	{
+		#if windows
+		if (value)
+			flags |= FileAttribute.TEMPORARY;
+		else
+			flags &= ~FileAttribute.TEMPORARY;
+		return value;
+		#else
+		return false;
+		#end
+	}
+
+	// Non Settables
+	public var isCompressed(get, never):Bool;
+
+	private function get_isCompressed():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.COMPRESSED) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	public var isDevice(get, never):Bool;
+
+	private function get_isDevice():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.DEVICE) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	public var isDirectory(get, never):Bool;
+
+	private function get_isDirectory():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.DIRECTORY) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	public var isEncrypted(get, never):Bool;
+
+	private function get_isEncrypted():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.ENCRYPTED) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	public var isReparsePoint(get, never):Bool;
+
+	private function get_isReparsePoint():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.REPARSE_POINT) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	public var isSparseFile(get, never):Bool;
+
+	private function get_isSparseFile():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.SPARSE_FILE) != 0;
+		#else
+		return false;
+		#end
+	}
+
+	// For checking (mainly)
+	public var isNothing(get, never):Bool;
+
+	private function get_isNothing():Bool
+	{
+		#if windows
+		return (flags & FileAttribute.NOTHING) == 0;
+		#else
+		return false;
+		#end
+	}
+
+	public function getValue():Int
+	{
+		return flags;
+	}
+}
diff --git a/source/funkin/backend/utils/NativeAPI.hx b/source/funkin/backend/utils/NativeAPI.hx
index 495403dc..d7bd6030 100644
--- a/source/funkin/backend/utils/NativeAPI.hx
+++ b/source/funkin/backend/utils/NativeAPI.hx
@@ -1,6 +1,7 @@
 package funkin.backend.utils;
 
 import funkin.backend.utils.native.*;
+import flixel.util.typeLimit.OneOfTwo;
 
 /**
  * Class for functions that talk to a lower level than haxe, such as message boxes, and more.
@@ -32,7 +33,7 @@ class NativeAPI {
 	/**
 	 * Gets the specified file's (or folder) attribute.
 	 */
-	public static function getFileAttribute(path:String, useAbsol:Bool = true):FileAttribute {
+	public static function getFileAttributeRaw(path:String, useAbsol:Bool = true):Int {
 		#if windows
 		if(useAbsol) path = sys.FileSystem.absolutePath(path);
 		return Windows.getFileAttribute(path);
@@ -41,6 +42,13 @@ class NativeAPI {
 		#end
 	}
 
+	/**
+	 * Gets the specified file's (or folder) attribute.
+	 */
+	public static function getFileAttribute(path:String, useAbsol:Bool = true):FileAttributeWrapper {
+		return new FileAttributeWrapper(getFileAttributeRaw(path, useAbsol));
+	}
+
 	/**
 	 * Sets the specified file's (or folder) attribute. If it fails, the return value is `0`.
 	 */
@@ -53,6 +61,21 @@ class NativeAPI {
 		#end
 	}
 
+	/**
+	 * Sets the specified file's (or folder) attribute. If it fails, the return value is `0`.
+	 */
+	public static function setFileAttributes(path:String, attrib:OneOfTwo<FileAttributeWrapper, Int>, useAbsol:Bool = true):Int {
+		#if windows
+		if(useAbsol) path = sys.FileSystem.absolutePath(path);
+		if(attrib is FileAttributeWrapper)
+			return Windows.setFileAttribute(path, cast(attrib, FileAttributeWrapper).flags);
+		else
+			return Windows.setFileAttribute(path, cast(attrib, Int));
+		#else
+		return 0;
+		#end
+	}
+
 	public static function setDarkMode(title:String, enable:Bool) {
 		#if windows
 		Windows.setDarkMode(title, enable);
@@ -137,7 +160,7 @@ class NativeAPI {
 	}
 }
 
-enum abstract FileAttribute(Int) {
+enum abstract FileAttribute(Int) from Int to Int {
 	// Settables
 	var ARCHIVE = 0x20;
 	var HIDDEN = 0x2;
@@ -155,6 +178,9 @@ enum abstract FileAttribute(Int) {
 	var ENCRYPTED = 0x4000;
 	var REPARSE_POINT = 0x400;
 	var SPARSE_FILE = 0x200;
+
+	// For checking (mainly)
+	var NOTHING = -1;
 }
 
 enum abstract ConsoleColor(Int) {

From 6715e2b1ccec52ef82be92362a90e18a1b2ad7da Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=8D=9A=7ENex?=
 <87421482+NexIsDumb@users.noreply.github.com>
Date: Tue, 18 Jun 2024 04:54:59 +0200
Subject: [PATCH 07/12] even cooler stuff for file attributes

---
 source/funkin/backend/system/MainState.hx     |  2 +-
 source/funkin/backend/utils/CoolUtil.hx       | 50 ++++++++++------
 .../backend/utils/FileAttributeWrapper.hx     | 57 ++++++++-----------
 source/funkin/backend/utils/NativeAPI.hx      | 45 ++++++++-------
 source/funkin/backend/utils/native/Windows.hx |  4 +-
 5 files changed, 85 insertions(+), 73 deletions(-)

diff --git a/source/funkin/backend/system/MainState.hx b/source/funkin/backend/system/MainState.hx
index d7238eec..14c40aff 100644
--- a/source/funkin/backend/system/MainState.hx
+++ b/source/funkin/backend/system/MainState.hx
@@ -66,6 +66,6 @@ class MainState extends FlxState {
 			betaWarningShown = true;
 		}
 
-		CoolUtil.safeSetAttribute('./.temp/', HIDDEN);
+		CoolUtil.safeAddAttributes('./.temp/', NativeAPI.FileAttribute.HIDDEN);
 	}
 }
\ No newline at end of file
diff --git a/source/funkin/backend/utils/CoolUtil.hx b/source/funkin/backend/utils/CoolUtil.hx
index 979662e0..63a910b1 100644
--- a/source/funkin/backend/utils/CoolUtil.hx
+++ b/source/funkin/backend/utils/CoolUtil.hx
@@ -6,6 +6,7 @@ import sys.FileSystem;
 import flixel.text.FlxText;
 import funkin.backend.utils.XMLUtil.TextFormat;
 import flixel.util.typeLimit.OneOfTwo;
+import flixel.util.typeLimit.OneOfThree;
 import flixel.tweens.FlxTween;
 import flixel.system.frontEnds.SoundFrontEnd;
 import flixel.sound.FlxSound;
@@ -99,18 +100,17 @@ class CoolUtil
 	}
 
 	/**
-	 * Sets an attribute to a file or a folder adding eventual missing folders in the path
-	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
+	 * Gets file attributes from a file or a folder adding eventual missing folders in the path
+	 * (WARNING: Only works on `windows` for now. On other platforms the attributes' value it's always going to be `0` -thanks to the wrapper you can also use `isNothing` for checking- but still creates eventual missing folders if the platforms allows it to).
 	 * @param path Path to the file or folder
-	 * @param attrib The attribute to set (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
 	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
-	 * @return The result code: 0 means that it failed setting
+	 * @return The attributes through the `FileAttributeWrapper`
 	 */
-	@:noUsing public static inline function safeSetAttribute(path:String, attrib:NativeAPI.FileAttribute, useAbsol:Bool = true) {
+	@:noUsing public static inline function safeGetAttributes(path:String, useAbsol:Bool = true):FileAttributeWrapper {
 		addMissingFolders(Path.directory(path));
 
-		var result = NativeAPI.setFileAttribute(path, attrib, useAbsol);
-		if(result == 0) Logs.trace('Failed to set attribute to $path with a code of: $result', WARNING);
+		var result = NativeAPI.getFileAttributes(path, useAbsol);
+		if(result.isNothing) Logs.trace('The file where it has been tried to get the attributes from, might be corrupted or inexistent (code: ${result.getValue()})', WARNING);
 		return result;
 	}
 
@@ -118,11 +118,12 @@ class CoolUtil
 	 * Sets file attributes to a file or a folder adding eventual missing folders in the path
 	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
 	 * @param path Path to the file or folder
-	 * @param attrib The attribute to set (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
+	 * @param attrib The attribute(s) to set (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
 	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
-	 * @return The result code: 0 means that it failed setting
+	 * @return The result code: `0` means that it failed setting
 	 */
-	 @:noUsing public static inline function safeSetAttributes(path:String, attrib:OneOfTwo<FileAttributeWrapper, Int>, useAbsol:Bool = true) {
+	@:noUsing public static inline function safeSetAttributes(path:String, attrib:OneOfThree<NativeAPI.FileAttribute, FileAttributeWrapper, Int>, useAbsol:Bool = true):Int {
+		// yes, i'm aware that FileAttribute is also an Int so need to include it too, but at least like this we don't have to make cast sometimes while passing the arguments  - Nex
 		addMissingFolders(Path.directory(path));
 
 		var result = NativeAPI.setFileAttributes(path, attrib, useAbsol);
@@ -131,21 +132,36 @@ class CoolUtil
 	}
 
 	/**
-	 * Gets the attributes of a file or a folder adding eventual missing folders in the path
+	 * Adds one (or more) file attributes to a file or a folder adding eventual missing folders in the path
 	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
 	 * @param path Path to the file or folder
+	 * @param attrib The attribute(s) to add (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
 	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
-	 * @return The result code: 0 means that it failed setting
+	 * @return The result code: `0` means that it failed setting
 	 */
-	 // uncomment if it crashes when using it normall
-	/*@:noUsing public static inline function safeGetAttribute(path:String, useAbsol:Bool = true) {
+	@:noUsing public static inline function safeAddAttributes(path:String, attrib:OneOfTwo<NativeAPI.FileAttribute, Int>, useAbsol:Bool = true):Int {
 		addMissingFolders(Path.directory(path));
 
-		var result = NativeAPI.getFileAttribute(path, useAbsol);
-		if(result == 0) Logs.trace('Failed to get attribute to $path with a code of: $result', WARNING);
+		var result = NativeAPI.addFileAttributes(path, attrib, useAbsol);
+		if(result == 0) Logs.trace('Failed to add attributes to $path with a code of: $result', WARNING);
 		return result;
-	}*/
+	}
 
+	/**
+	 * Removes one (or more) file attributes to a file or a folder adding eventual missing folders in the path
+	 * (WARNING: Only works on `windows` for now. On other platforms the return code it's always going to be `0` but still creates eventual missing folders if the platforms allows it to).
+	 * @param path Path to the file or folder
+	 * @param attrib The attribute(s) to remove (WARNING: There are some non settable attributes, such as the `COMPRESSED` one)
+	 * @param useAbsol If it should use the absolute path (By default it's `true` but if it's `false` you can use files outside from this program's directory for example)
+	 * @return The result code: `0` means that it failed setting
+	 */
+	@:noUsing public static inline function safeRemoveAttributes(path:String, attrib:OneOfTwo<NativeAPI.FileAttribute, Int>, useAbsol:Bool = true):Int {
+		addMissingFolders(Path.directory(path));
+
+		var result = NativeAPI.removeFileAttributes(path, attrib, useAbsol);
+		if(result == 0) Logs.trace('Failed to remove attributes to $path with a code of: $result', WARNING);
+		return result;
+	}
 
 	/**
 	 * Creates eventual missing folders to the specified `path`
diff --git a/source/funkin/backend/utils/FileAttributeWrapper.hx b/source/funkin/backend/utils/FileAttributeWrapper.hx
index 0b526223..7d7e26f4 100644
--- a/source/funkin/backend/utils/FileAttributeWrapper.hx
+++ b/source/funkin/backend/utils/FileAttributeWrapper.hx
@@ -1,5 +1,6 @@
 package funkin.backend.utils;
 
+import flixel.util.FlxStringUtil;
 import funkin.backend.utils.NativeAPI.FileAttribute;
 
 /**
@@ -7,15 +8,32 @@ import funkin.backend.utils.NativeAPI.FileAttribute;
  */
 class FileAttributeWrapper
 {
-	public var flags:Int;
+	private var flags:Int;
+	inline public function getValue():Int
+	{
+		return flags;
+	}
 
 	public function new(flags:Int)
 	{
-		this.flags = flags;
+		this.flags = flags == -1 ? 0 : flags;
 	}
 
-	public var isArchived(get, set):Bool;
+	/**
+	 * Returns a string representation of the attributes.
+	 */
+	inline public function toString():String
+	{
+		return FlxStringUtil.getDebugString([
+			for (field in Reflect.fields(this))
+			{
+				LabelValuePair.weak(field, Reflect.getProperty(this, field));
+			}
+		]);
+	}
 
+	// Settables
+	public var isArchived(get, set):Bool;
 	private function get_isArchived():Bool
 	{
 		#if windows
@@ -24,7 +42,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isArchived(value:Bool):Bool
 	{
 		#if windows
@@ -39,7 +56,6 @@ class FileAttributeWrapper
 	}
 
 	public var isHidden(get, set):Bool;
-
 	private function get_isHidden():Bool
 	{
 		#if windows
@@ -48,7 +64,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isHidden(value:Bool):Bool
 	{
 		#if windows
@@ -63,7 +78,6 @@ class FileAttributeWrapper
 	}
 
 	public var isNormal(get, set):Bool;
-
 	private function get_isNormal():Bool
 	{
 		#if windows
@@ -72,7 +86,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isNormal(value:Bool):Bool
 	{
 		#if windows
@@ -87,7 +100,6 @@ class FileAttributeWrapper
 	}
 
 	public var isNotContentIndexed(get, set):Bool;
-
 	private function get_isNotContentIndexed():Bool
 	{
 		#if windows
@@ -96,7 +108,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isNotContentIndexed(value:Bool):Bool
 	{
 		#if windows
@@ -111,7 +122,6 @@ class FileAttributeWrapper
 	}
 
 	public var isOffline(get, set):Bool;
-
 	private function get_isOffline():Bool
 	{
 		#if windows
@@ -120,7 +130,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isOffline(value:Bool):Bool
 	{
 		#if windows
@@ -135,7 +144,6 @@ class FileAttributeWrapper
 	}
 
 	public var isReadOnly(get, set):Bool;
-
 	private function get_isReadOnly():Bool
 	{
 		#if windows
@@ -144,7 +152,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isReadOnly(value:Bool):Bool
 	{
 		#if windows
@@ -159,7 +166,6 @@ class FileAttributeWrapper
 	}
 
 	public var isSystem(get, set):Bool;
-
 	private function get_isSystem():Bool
 	{
 		#if windows
@@ -168,7 +174,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isSystem(value:Bool):Bool
 	{
 		#if windows
@@ -183,7 +188,6 @@ class FileAttributeWrapper
 	}
 
 	public var isTemporary(get, set):Bool;
-
 	private function get_isTemporary():Bool
 	{
 		#if windows
@@ -192,7 +196,6 @@ class FileAttributeWrapper
 		return false;
 		#end
 	}
-
 	private function set_isTemporary(value:Bool):Bool
 	{
 		#if windows
@@ -208,7 +211,6 @@ class FileAttributeWrapper
 
 	// Non Settables
 	public var isCompressed(get, never):Bool;
-
 	private function get_isCompressed():Bool
 	{
 		#if windows
@@ -219,7 +221,6 @@ class FileAttributeWrapper
 	}
 
 	public var isDevice(get, never):Bool;
-
 	private function get_isDevice():Bool
 	{
 		#if windows
@@ -230,7 +231,6 @@ class FileAttributeWrapper
 	}
 
 	public var isDirectory(get, never):Bool;
-
 	private function get_isDirectory():Bool
 	{
 		#if windows
@@ -241,7 +241,6 @@ class FileAttributeWrapper
 	}
 
 	public var isEncrypted(get, never):Bool;
-
 	private function get_isEncrypted():Bool
 	{
 		#if windows
@@ -252,7 +251,6 @@ class FileAttributeWrapper
 	}
 
 	public var isReparsePoint(get, never):Bool;
-
 	private function get_isReparsePoint():Bool
 	{
 		#if windows
@@ -263,7 +261,6 @@ class FileAttributeWrapper
 	}
 
 	public var isSparseFile(get, never):Bool;
-
 	private function get_isSparseFile():Bool
 	{
 		#if windows
@@ -273,20 +270,14 @@ class FileAttributeWrapper
 		#end
 	}
 
-	// For checking (mainly)
+	// For checking
 	public var isNothing(get, never):Bool;
-
 	private function get_isNothing():Bool
 	{
 		#if windows
-		return (flags & FileAttribute.NOTHING) == 0;
+		return flags == 0;
 		#else
-		return false;
+		return true;
 		#end
 	}
-
-	public function getValue():Int
-	{
-		return flags;
-	}
 }
diff --git a/source/funkin/backend/utils/NativeAPI.hx b/source/funkin/backend/utils/NativeAPI.hx
index d7bd6030..78f0b38d 100644
--- a/source/funkin/backend/utils/NativeAPI.hx
+++ b/source/funkin/backend/utils/NativeAPI.hx
@@ -2,6 +2,7 @@ package funkin.backend.utils;
 
 import funkin.backend.utils.native.*;
 import flixel.util.typeLimit.OneOfTwo;
+import flixel.util.typeLimit.OneOfThree;
 
 /**
  * Class for functions that talk to a lower level than haxe, such as message boxes, and more.
@@ -31,46 +32,53 @@ class NativeAPI {
 	}
 
 	/**
-	 * Gets the specified file's (or folder) attribute.
+	 * Gets the specified file's (or folder) attributes.
 	 */
-	public static function getFileAttributeRaw(path:String, useAbsol:Bool = true):Int {
+	public static function getFileAttributesRaw(path:String, useAbsol:Bool = true):Int {
 		#if windows
 		if(useAbsol) path = sys.FileSystem.absolutePath(path);
-		return Windows.getFileAttribute(path);
+		return Windows.getFileAttributes(path);
 		#else
-		return NORMAL;
+		return -1;
 		#end
 	}
 
 	/**
-	 * Gets the specified file's (or folder) attribute.
+	 * Gets the specified file's (or folder) attributes and passes it to `FileAttributeWrapper`.
 	 */
-	public static function getFileAttribute(path:String, useAbsol:Bool = true):FileAttributeWrapper {
-		return new FileAttributeWrapper(getFileAttributeRaw(path, useAbsol));
+	public static function getFileAttributes(path:String, useAbsol:Bool = true):FileAttributeWrapper {
+		return new FileAttributeWrapper(getFileAttributesRaw(path, useAbsol));
 	}
 
 	/**
-	 * Sets the specified file's (or folder) attribute. If it fails, the return value is `0`.
+	 * Sets the specified file's (or folder) attributes. If it fails, the return value is `0`.
 	 */
-	public static function setFileAttribute(path:String, attrib:FileAttribute, useAbsol:Bool = true):Int {
+	public static function setFileAttributes(path:String, attrib:OneOfThree<NativeAPI.FileAttribute, FileAttributeWrapper, Int>, useAbsol:Bool = true):Int {
 		#if windows
 		if(useAbsol) path = sys.FileSystem.absolutePath(path);
-		return Windows.setFileAttribute(path, attrib);
+		return Windows.setFileAttributes(path, attrib is FileAttributeWrapper ? cast(attrib, FileAttributeWrapper).getValue() : cast(attrib, Int));
 		#else
 		return 0;
 		#end
 	}
 
 	/**
-	 * Sets the specified file's (or folder) attribute. If it fails, the return value is `0`.
+	 * Removes from the specified file's (or folder) one (or more) specific attribute.
 	 */
-	public static function setFileAttributes(path:String, attrib:OneOfTwo<FileAttributeWrapper, Int>, useAbsol:Bool = true):Int {
+	public static function addFileAttributes(path:String, attrib:OneOfTwo<NativeAPI.FileAttribute, Int>, useAbsol:Bool = true):Int {
 		#if windows
-		if(useAbsol) path = sys.FileSystem.absolutePath(path);
-		if(attrib is FileAttributeWrapper)
-			return Windows.setFileAttribute(path, cast(attrib, FileAttributeWrapper).flags);
-		else
-			return Windows.setFileAttribute(path, cast(attrib, Int));
+		return setFileAttributes(path, getFileAttributesRaw(path, useAbsol) | cast(attrib, Int), useAbsol);
+		#else
+		return 0;
+		#end
+	}
+
+	/**
+	 * Removes from the specified file's (or folder) one (or more) specific attribute.
+	 */
+	public static function removeFileAttributes(path:String, attrib:OneOfTwo<NativeAPI.FileAttribute, Int>, useAbsol:Bool = true):Int {
+		#if windows
+		return setFileAttributes(path, getFileAttributesRaw(path, useAbsol) & ~cast(attrib, Int), useAbsol);
 		#else
 		return 0;
 		#end
@@ -178,9 +186,6 @@ enum abstract FileAttribute(Int) from Int to Int {
 	var ENCRYPTED = 0x4000;
 	var REPARSE_POINT = 0x400;
 	var SPARSE_FILE = 0x200;
-
-	// For checking (mainly)
-	var NOTHING = -1;
 }
 
 enum abstract ConsoleColor(Int) {
diff --git a/source/funkin/backend/utils/native/Windows.hx b/source/funkin/backend/utils/native/Windows.hx
index 5b03b904..a536c75f 100644
--- a/source/funkin/backend/utils/native/Windows.hx
+++ b/source/funkin/backend/utils/native/Windows.hx
@@ -157,7 +157,7 @@ class Windows {
 	@:functionCode('
 		return GetFileAttributes(path);
 	')
-	public static function getFileAttribute(path:String):FileAttribute
+	public static function getFileAttributes(path:String):FileAttribute
 	{
 		return NORMAL;
 	}
@@ -165,7 +165,7 @@ class Windows {
 	@:functionCode('
 		return SetFileAttributes(path, attrib);
 	')
-	public static function setFileAttribute(path:String, attrib:FileAttribute):Int
+	public static function setFileAttributes(path:String, attrib:FileAttribute):Int
 	{
 		return 0;
 	}

From 5a5a25a255ea29ecc39f10959746c9adbf01a934 Mon Sep 17 00:00:00 2001
From: ~ Care <126361810+usb-port-2@users.noreply.github.com>
Date: Wed, 19 Jun 2024 00:25:33 +0100
Subject: [PATCH 08/12] super important (#321)

the game would be broken without this, thank you   - @CharlesCatYT 19/06/2024
---
 source/funkin/backend/system/Conductor.hx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source/funkin/backend/system/Conductor.hx b/source/funkin/backend/system/Conductor.hx
index 8db60b36..bd5fca4c 100644
--- a/source/funkin/backend/system/Conductor.hx
+++ b/source/funkin/backend/system/Conductor.hx
@@ -168,7 +168,7 @@ class Conductor
 			// update conductor
 			songPosition = lastSongPos;
 		} else {
-			songPosition = songPosition + songOffset + elapsed * 1000;
+			songPosition += songOffset + elapsed * 1000;
 		}
 	}
 

From d79959d0504562b952d04d5e3898847fcf45f0a6 Mon Sep 17 00:00:00 2001
From: ~ Care <126361810+usb-port-2@users.noreply.github.com>
Date: Mon, 24 Jun 2024 23:07:37 +0100
Subject: [PATCH 09/12] Propose changes (#324)

---
 source/funkin/game/PlayState.hx | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/source/funkin/game/PlayState.hx b/source/funkin/game/PlayState.hx
index 67d5d822..e97bee1a 100644
--- a/source/funkin/game/PlayState.hx
+++ b/source/funkin/game/PlayState.hx
@@ -359,11 +359,19 @@ class PlayState extends MusicBeatState
 	 * Camera zoom at which the game lerps to.
 	 */
 	public var defaultCamZoom:Float = 1.05;
-
+	/**
+	 * Speed at which the game camera zoom lerps to.
+	 */
+	public var camGameZoomLerp:Float = 0.05;
+	
 	/**
 	 * Camera zoom at which the hud lerps to.
 	 */
 	public var defaultHudZoom:Float = 1.0;
+	/**
+	 * Speed at which the hud camera zoom lerps to.
+	 */
+	public var camHUDZoomLerp:Float = 0.05;
 
 	/**
 	 * Zoom for the pixel assets.
@@ -1290,8 +1298,8 @@ class PlayState extends MusicBeatState
 
 		if (camZooming)
 		{
-			FlxG.camera.zoom = lerp(FlxG.camera.zoom, defaultCamZoom, 0.05);
-			camHUD.zoom = lerp(camHUD.zoom, defaultHudZoom, 0.05);
+			FlxG.camera.zoom = lerp(FlxG.camera.zoom, defaultCamZoom, camGameZoomLerp);
+			camHUD.zoom = lerp(camHUD.zoom, defaultHudZoom, camHUDZoomLerp);
 		}
 
 		// RESET = Quick Game Over Screen

From ce19a63225df3eba0135959f14e83ec4731b602f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=8D=9A=7ENex?=
 <87421482+NexIsDumb@users.noreply.github.com>
Date: Thu, 27 Jun 2024 20:39:20 +0200
Subject: [PATCH 10/12] fixing two character offset bugs rq

---
 source/funkin/editors/character/CharacterEditor.hx | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/source/funkin/editors/character/CharacterEditor.hx b/source/funkin/editors/character/CharacterEditor.hx
index 10fa4efa..95da4989 100644
--- a/source/funkin/editors/character/CharacterEditor.hx
+++ b/source/funkin/editors/character/CharacterEditor.hx
@@ -348,10 +348,6 @@ class CharacterEditor extends UIState {
 	}
 
 	function buildCharacter():String {
-		if (character.isPlayer != character.playerOffsets) {
-			character.switchOffset('singLEFT', 'singRIGHT');
-			character.switchOffset('singLEFTmiss', 'singRIGHTmiss');
-		}
 		var charXML:Xml = character.buildXML([
 			for (button in characterAnimsWindow.buttons.members)
 				button.anim
@@ -588,6 +584,12 @@ class CharacterEditor extends UIState {
 	function changeOffset(anim:String, change:FlxPoint, addtoUndo:Bool = true) {
 		if (character.getNameList().length == 0) return;
 
+		var animData = character.animDatas.get(anim);
+		if (animData != null) {
+			animData.x += change.x;
+			animData.y += change.y;
+		}
+
 		character.animOffsets.set(anim, character.getAnimOffset(anim) + change);
 		for (i in characterAnimsWindow.buttons.members)
 			if (i.anim == anim)

From 33275573278891be7801bc0cf6368ce59483443f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=8D=9A=7ENex?=
 <87421482+NexIsDumb@users.noreply.github.com>
Date: Fri, 28 Jun 2024 01:16:59 +0200
Subject: [PATCH 11/12] linux??

---
 .github/workflows/linux.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/linux.yml b/.github/workflows/linux.yml
index 76adeded..8ac80b6a 100644
--- a/.github/workflows/linux.yml
+++ b/.github/workflows/linux.yml
@@ -7,7 +7,7 @@ jobs:
   build:
     name: Linux Build
     permissions: write-all
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-latest
     steps:
       - name: Pulling the source
         uses: actions/checkout@v2

From 1e8149e0944dcab31d6196ba47e681da4e69d52d Mon Sep 17 00:00:00 2001
From: "Rodney, An Imaginative Person"
 <59747809+RodneyAnImaginativePerson@users.noreply.github.com>
Date: Sun, 30 Jun 2024 18:02:26 -0400
Subject: [PATCH 12/12] Update Character.hx (#330)

---
 source/funkin/game/Character.hx | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/source/funkin/game/Character.hx b/source/funkin/game/Character.hx
index c2faaafc..286ff86e 100644
--- a/source/funkin/game/Character.hx
+++ b/source/funkin/game/Character.hx
@@ -238,6 +238,8 @@ class Character extends FunkinSprite implements IBeatReceiver implements IOffset
 	}
 
 	public override function destroy() {
+		script.call('destroy');
+		script.destroy();
 		super.destroy();
 
 		cameraOffset.put();