Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EMCO Creates the wrong dir for loggin. #42

Closed
MorquinDevlar opened this issue Oct 4, 2022 · 8 comments
Closed

EMCO Creates the wrong dir for loggin. #42

MorquinDevlar opened this issue Oct 4, 2022 · 8 comments

Comments

@MorquinDevlar
Copy link

MorquinDevlar commented Oct 4, 2022

It may be a direct Mudlet problem, but it didn't appear before I started using EMCO to created tabbed chat capture.

It looks like a wrong directory is created for loggin, if one doesn't already exits.

EMCO wants to log in ...profile_name/_E/, but it creates a dir like ...profile_name/|E/ if the former doesn't exist. It still tries to write to _E, but can't until you manually create it.

Screenshot 2022-10-04 at 11 39 26

Even when I create the _E folder, it still creates the |E folder, but then proceeds to write to _E

@demonnic
Copy link
Owner

demonnic commented Oct 4, 2022

Hmm, I'll take a look. This is with the latest MDK? I recently made a change around this and it was testing out ok but perhaps you found an edge case I missed.

@MorquinDevlar
Copy link
Author

It's the latest version yes. And I am on OSX btw.

@demonnic
Copy link
Owner

demonnic commented Oct 4, 2022

Does this still occur after you gave the EMCO a name? the |E is meant to be templated out to the name of the EMCO in question, and I wonder if the two might not be related.

@MorquinDevlar
Copy link
Author

Just tested it, and yes. Even with names it creates the wrong dir.

@demonnic
Copy link
Owner

demonnic commented Oct 4, 2022

I wonder why I wasn't having this problem in testing. Something's gone wrong with the templating used to derive the directory, clearly. But I'm not sure what yet. In the meantime, try 2.7.0 and see if that works properly?

@demonnic
Copy link
Owner

demonnic commented Oct 5, 2022

MDK.zip
this is the mpackage of what I believe will fix the issue for you. If it does, pls let me know? I have one more fix I want to put in place before I do another release.

@demonnic
Copy link
Owner

demonnic commented Oct 5, 2022

MDK.zip
never mind, this one should fix it. I see what happened now.

@MorquinDevlar
Copy link
Author

Screenshot 2022-10-05 at 19 25 47

Worked perfectly! Thanks for the 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 a pull request may close this issue.

2 participants