Started by GitHub push by blowfishpro32267c64af949627f55fe740296773b4d72ef88c7e267c64af949627f55fe740296773b4d72ef88c7erefs/remotes/origin/dev267c64af949627f55fe740296773b4d72ef88c7e267c64af949627f55fe740296773b4d72ef88c7erefs/remotes/origin/dev267c64af949627f55fe740296773b4d72ef88c7e267c64af949627f55fe740296773b4d72ef88c7erefs/remotes/origin/devhttps://github.com/sarbian/ModuleManager.githttps://ksp.sarbian.com/jenkins/user/videogamer0907/my-views/view/all/job/ModuleManager-Dev/32/artifacthttps://ksp.sarbian.com/jenkins/user/videogamer0907/my-views/view/all/job/ModuleManager-Dev/changeshttps://ksp.sarbian.com/jenkins/user/videogamer0907/my-views/view/all/job/ModuleManager-Dev/32/https://ksp.sarbian.com/jenkins/user/videogamer0907/my-views/view/all/job/ModuleManager-Dev/32/testReportModuleManager.dllModuleManager.dllModuleManager.dllfalse#3267536498ModuleManager-Dev #3232falsefalse32224SUCCESS1524634226008https://ksp.sarbian.com/jenkins/user/videogamer0907/my-views/view/all/job/ModuleManager-Dev/32/ModuleManagerTests/ModuleManagerTests.csprojModuleManager/MMPatchLoader.csModuleManagerTests/MMPatchLoaderTest.csc28744139c01a32caf2c98e8749ff9ba5276c3d51524206409000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comFix value assignment with * indexer Broken in #111 - probably an unusual case but it would have worked before. Added tests to ensure that this fixes it. Tests are not and will probably never cover all of MMPatchLoader.ModifyNode but useful to add bugfix cases here as they occur. 2018-04-19 23:40:09 -0700c28744139c01a32caf2c98e8749ff9ba5276c3d5Fix value assignment with * indexereditModuleManager/MMPatchLoader.csaddModuleManagerTests/MMPatchLoaderTest.cseditModuleManagerTests/ModuleManagerTests.csprojTestUtils/UrlBuilder.csModuleManagerTests/NeedsCheckerTest.cs0aa64b5f002c42400ffc3ef7f5499370f79a03131524298081000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comReflection fields should be readonly 2018-04-21 01:08:01 -07000aa64b5f002c42400ffc3ef7f5499370f79a0313Reflection fields should be readonlyeditModuleManagerTests/NeedsCheckerTest.cseditTestUtils/UrlBuilder.csTestUtils/UrlBuilder.csTestUtilsTests/UrlBuilderTest.csae2a14f9154564360f07a6ef9d41ebc5add23eb61524298140000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comCreate special GameData subdirectory It's special 2018-04-21 01:09:00 -0700ae2a14f9154564360f07a6ef9d41ebc5add23eb6Create special GameData subdirectoryeditTestUtilsTests/UrlBuilderTest.cseditTestUtils/UrlBuilder.csModuleManager/NeedsChecker.csModuleManagerTests/NeedsCheckerTest.cs4a7e3c8a455084f1a05ec187819c46db66a2b1d31524298614000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comAllow checking needs against directories If the needs string contains a / it will check for a directory with that path in GameData. Notes: * PluginData folders are excluded * Leading and trailing slashes are allowed * Multiple slashes together will be treated as a single slash * Comaprison is case sensitive 2018-04-21 01:16:54 -07004a7e3c8a455084f1a05ec187819c46db66a2b1d3Allow checking needs against directorieseditModuleManager/NeedsChecker.cseditModuleManagerTests/NeedsCheckerTest.csModuleManager/MMPatchLoader.cs6c73797fa180d354ffabf9cc39500af6d3c625ab1524634214000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake patch stopwatch local It's not needed outside of ProcessPatch(), which contains everything it needs to time 2018-04-24 22:30:14 -07006c73797fa180d354ffabf9cc39500af6d3c625abMake patch stopwatch localeditModuleManager/MMPatchLoader.csModuleManager/MMPatchLoader.cs267c64af949627f55fe740296773b4d72ef88c7e1524634214000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake useCache local Really only needed for a few lines in ProcessPatch() 2018-04-24 22:30:14 -0700267c64af949627f55fe740296773b4d72ef88c7eMake useCache localeditModuleManager/MMPatchLoader.csgithttps://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong