Skip to content

reflection fix - #8

Open
Duskdotso wants to merge 2 commits into
GorillaTagModdingHub:mainfrom
Duskdotso:main
Open

reflection fix#8
Duskdotso wants to merge 2 commits into
GorillaTagModdingHub:mainfrom
Duskdotso:main

Conversation

@Duskdotso

Copy link
Copy Markdown

this is the thing i did to fix it dying with melonloader stuff, likely could do more and make it less stupid looking, but my stuff is very limited rn

@LapisGit LapisGit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change seems to work, but prevents the built in "Outfits" wardrobe section from loading.

@Duskdotso

Copy link
Copy Markdown
Author

@LapisGit oh, didnt notice that. idk how to fix that personally (at least not at the moment). D:

@LapisGit

Copy link
Copy Markdown
Member

all good, ill take a look at it

@Duskdotso

Copy link
Copy Markdown
Author

okie! 👍

@LapisGit

Copy link
Copy Markdown
Member

@Duskdotso does the latest commit i sent seem to be fine? i think the reflection fix should still work

@Duskdotso

Copy link
Copy Markdown
Author

@LapisGit nope, that broke the wardrobe on my end. ill also see if i can fix a new error that i just got too. got this error:

Stack trace:
GorillaLibrary.Behaviours.WardrobeController.OnOutfitTextUpdate () (at C:/Users/dusk/Downloads/GorillaLibrary-main/GorillaLibrary-main/GorillaLibrary/Behaviours/WardrobeController.cs:445)
GorillaLibrary.Patches.CosmeticWardrobePatches.OutfitTextUpdatePatch (CosmeticWardrobe __instance) (at C:/Users/dusk/Downloads/GorillaLibrary-main/GorillaLibrary-main/GorillaLibrary/Patches/CosmeticWardrobePatches.cs:99)
(wrapper dynamic-method) CosmeticWardrobe.DMD<CosmeticWardrobe::UpdateOutfitButtons>(CosmeticWardrobe)
(wrapper dynamic-method) CosmeticWardrobe.DMD<CosmeticWardrobe::HandleCosmeticsUpdated>(CosmeticWardrobe)
GorillaNetworking.CosmeticsController+<>c__DisplayClass232_0.<GetCosmeticsPlayFabCatalogData>b__2 (PlayFab.ClientModels.GetCatalogItemsResult result2) (at <13ed87914c3a4539b93b34eb8f8f3fd5>:0)
PlayFab.Internal.PlayFabHttp+<>c__DisplayClass23_0`1[TResult].<_MakeApiCall>b__1 () (at <9decd3d7936442d2b2ce8b96a25374b4>:0)
PlayFab.Internal.PlayFabUnityHttp.OnResponse (System.String response, PlayFab.Internal.CallRequestContainer reqContainer) (at <9decd3d7936442d2b2ce8b96a25374b4>:0)
UnityEngine.Debug:LogException(Exception)
PlayFab.Internal.PlayFabUnityHttp:OnResponse(String, CallRequestContainer)
PlayFab.Internal.<Post>d__13:MoveNext()
UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)

@Duskdotso

Copy link
Copy Markdown
Author

think i found the issue. line 108 in plugin.cs, its going through all the attributes, finding a melonloader attribute, and throwing an error since it doesnt know what it is (i think).

@LapisGit

Copy link
Copy Markdown
Member

Hm, alright. I'll take a look later if you don't find a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants