@quenty/camera 7.1.0 → 7.2.1-canary.276.672e37b.0

package/CHANGELOG.md

@@ -3,6 +3,25 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [7.2.1-canary.276.672e37b.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/camera@7.2.0...@quenty/camera@7.2.1-canary.276.672e37b.0) (2022-07-31)
+
+**Note:** Version bump only for package @quenty/camera
+
+
+
+
+
+# [7.2.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/camera@7.1.0...@quenty/camera@7.2.0) (2022-07-02)
+
+
+### Bug Fixes
+
+* Can clean up services properly ([eb45e03](https://github.com/Quenty/NevermoreEngine/commit/eb45e03ce2897b18f1ae460974bf2bbb9e27cb97))
+
+
+
+
+
 # [7.1.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/camera@7.0.0...@quenty/camera@7.1.0) (2022-06-21)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/camera",
-  "version": "7.1.0",
+  "version": "7.2.1-canary.276.672e37b.0",
   "description": "Quenty's camera system for Roblox",
   "keywords": [
     "Roblox",
@@ -25,23 +25,23 @@
     "Quenty"
   ],
   "dependencies": {
-    "@quenty/acceltween": "^2.2.0",
-    "@quenty/baseobject": "^5.0.0",
-    "@quenty/cframeutils": "^4.0.0",
-    "@quenty/cubicspline": "^5.0.0",
-    "@quenty/inputobjectutils": "^3.0.0",
-    "@quenty/loader": "^5.0.0",
-    "@quenty/maid": "^2.3.0",
-    "@quenty/math": "^2.2.0",
-    "@quenty/qframe": "^5.1.0",
-    "@quenty/servicebag": "^5.0.0",
-    "@quenty/spring": "^5.0.0"
+    "@quenty/acceltween": "2.2.0",
+    "@quenty/baseobject": "5.0.1-canary.276.672e37b.0",
+    "@quenty/cframeutils": "4.0.0",
+    "@quenty/cubicspline": "5.0.0",
+    "@quenty/inputobjectutils": "3.0.0",
+    "@quenty/loader": "5.0.0",
+    "@quenty/maid": "2.3.1-canary.276.672e37b.0",
+    "@quenty/math": "2.2.0",
+    "@quenty/qframe": "5.1.1-canary.276.672e37b.0",
+    "@quenty/servicebag": "5.0.1-canary.276.672e37b.0",
+    "@quenty/spring": "5.0.0"
   },
   "devDependencies": {
-    "@quenty/camerastoryutils": "^5.1.0"
+    "@quenty/camerastoryutils": "5.1.1-canary.276.672e37b.0"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "c8732cc5dea767b3ff362db43137e2a16da7bc0d"
+  "gitHead": "672e37b9bbb8b2d4c4c6c69ed2647335be0bc469"
 }

package/src/Client/CameraStackService.lua

@@ -15,6 +15,7 @@ local CustomCameraEffect = require("CustomCameraEffect")
 local DefaultCamera = require("DefaultCamera")
 local ImpulseCamera = require("ImpulseCamera")
 local ServiceBag = require("ServiceBag")
+local Maid = require("Maid")
 
 assert(RunService:IsClient(), "[CameraStackService] - Only require CameraStackService on client")
 
@@ -27,18 +28,23 @@ local CameraStackService = {}
 function CameraStackService:Init(serviceBag)
 	assert(ServiceBag.isServiceBag(serviceBag), "Not a valid service bag")
 
+	self._maid = Maid.new()
+	self._key = HttpService:GenerateGUID(false)
+
 	self._stack = {}
 	self._disabledSet = {}
 
 	-- Initialize default cameras
 	self._rawDefaultCamera = DefaultCamera.new()
+	self._maid:GiveTask(self._rawDefaultCamera)
+
 	self._impulseCamera = ImpulseCamera.new()
 	self._defaultCamera = (self._rawDefaultCamera + self._impulseCamera):SetMode("Relative")
 
 	-- Add camera to stack
 	self:Add(self._defaultCamera)
 
-	RunService:BindToRenderStep("CameraStackUpdateInternal", Enum.RenderPriority.Camera.Value + 75, function()
+	RunService:BindToRenderStep("CameraStackUpdateInternal" .. self._key, Enum.RenderPriority.Camera.Value + 75, function()
 		debug.profilebegin("CameraStackUpdate")
 
 		if next(self._disabledSet) then
@@ -52,6 +58,10 @@ function CameraStackService:Init(serviceBag)
 
 		debug.profileend()
 	end)
+
+	self._maid:GiveTask(function()
+		RunService:UnbindFromRenderStep("CameraStackUpdateInternal" .. self._key)
+	end)
 end
 
 function CameraStackService:Start()
@@ -271,4 +281,8 @@ function CameraStackService:Add(state)
 	table.insert(self._stack, state)
 end
 
+function CameraStackService:Destroy()
+	self._maid:DoCleaning()
+end
+
 return CameraStackService

package/src/Client/Effects/DefaultCamera.lua

@@ -10,6 +10,7 @@ local require = require(script.Parent.loader).load(script)
 
 local RunService = game:GetService("RunService")
 local Workspace = game:GetService("Workspace")
+local HttpService = game:GetService("HttpService")
 
 local CameraState = require("CameraState")
 local SummedCamera = require("SummedCamera")
@@ -25,6 +26,7 @@ DefaultCamera.ClassName = "DefaultCamera"
 function DefaultCamera.new()
 	local self = setmetatable({}, DefaultCamera)
 
+	self._key = HttpService:GenerateGUID(false)
 	self._cameraState = CameraState.new(Workspace.CurrentCamera)
 
 	return self
@@ -54,11 +56,11 @@ end
 	:::
 ]=]
 function DefaultCamera:BindToRenderStep()
-	RunService:BindToRenderStep("DefaultCamera_Preupdate", Enum.RenderPriority.Camera.Value-2, function()
+	RunService:BindToRenderStep("DefaultCamera_Preupdate" .. self._key, Enum.RenderPriority.Camera.Value-2, function()
 		self._cameraState:Set(Workspace.CurrentCamera)
 	end)
 
-	RunService:BindToRenderStep("DefaultCamera_PostUpdate", Enum.RenderPriority.Camera.Value+2, function()
+	RunService:BindToRenderStep("DefaultCamera_PostUpdate" .. self._key, Enum.RenderPriority.Camera.Value+2, function()
 		self._cameraState = CameraState.new(Workspace.CurrentCamera)
 	end)
 
@@ -69,8 +71,15 @@ end
 	Unbinds the camera from the RunService
 ]=]
 function DefaultCamera:UnbindFromRenderStep()
-	RunService:UnbindFromRenderStep("DefaultCamera_Preupdate")
-	RunService:UnbindFromRenderStep("DefaultCamera_PostUpdate")
+	RunService:UnbindFromRenderStep("DefaultCamera_Preupdate" .. self._key)
+	RunService:UnbindFromRenderStep("DefaultCamera_PostUpdate" .. self._key)
+end
+
+--[=[
+	Cleans up the binding
+]=]
+function DefaultCamera:Destroy()
+	self:UnbindFromRenderStep()
 end
 
 --[=[

package/test/default.project.json

@@ -1,10 +1,20 @@
 {
-  "name": "cameratest",
+  "name": "CameraTest",
   "tree": {
     "$className": "DataModel",
     "ServerScriptService": {
       "camera": {
         "$path": ".."
+      },
+      "Script": {
+        "$path": "scripts/Server"
+      }
+    },
+    "StarterPlayer": {
+      "StarterPlayerScripts": {
+        "Main": {
+          "$path": "scripts/Client"
+        }
       }
     }
   }

package/test/scripts/Client/ClientMain.client.lua

@@ -0,0 +1,11 @@
+--[[
+	@class ClientMain
+]]
+
+local packages = game:GetService("ReplicatedStorage"):WaitForChild("Packages")
+
+local serviceBag = require(packages.ServiceBag).new()
+serviceBag:GetService(require(packages.CameraStackService))
+
+serviceBag:Init()
+serviceBag:Start()

package/test/scripts/Server/ServerMain.server.lua

@@ -0,0 +1,8 @@
+--[[
+	@class ServerMain
+]]
+
+local ServerScriptService = game:GetService("ServerScriptService")
+
+local loader = ServerScriptService:FindFirstChild("LoaderUtils", true).Parent
+require(loader).bootstrapGame(ServerScriptService.camera)