"fix files and folders and permissions" not working and some other stuff about kkmanager HOT 13 CLOSED

What happens if you run this through your system's Run dialog? Does it show the same error? It assumes you have a directory D:\test
cmd /C echo off & cls & title Fixing permissions... & takeown /F "D:\test" /R /SKIPSL /D Y & icacls "D:\test" /grant everyone:F /T /C /L & pause
Also, what is the full path of your game's directory?

The menu got removed to avoid confusion.

from kkmanager.

The method grew into a monstrosity at the end of it https://github.com/IllusionMods/KKManager/blob/master/KKManager.Core/Util/ProcessTools.cs#L57

Yay for different system locales...

from kkmanager.

There is no system for overwriting cards currently, it'll have to be added.

The move command doesn't do anything either as well as gender change.

Those are not implemented yet.

takeown command displays this error and doesn't do anything else

What version of Windows are you running and exactly what do you do to trigger the issue?

from kkmanager.

If it's not implemented, why not hiding these options in the release version? I mean, it would avoid confusing users.

I'm on windows 10. To trigger the error, i just have to click on Tools / Fixes / Fix file and folder permissions then accept admin permissions. Then the terminal window appears and displays this error.

from kkmanager.

Can you check if the latest release still has the same issue with the permission fix?

from kkmanager.

Updated to 0.12.1 and still having the same error on that particular thing.
Except that it mentions Y and /D and not y and /d

(also the right click on a card doesn't show a menu anymore. Is this intended?)
(also also, awesome work on the program, i noticed i don't need to refresh manually when cards are displayed as large icons and they also seem to load faster ^^)

from kkmanager.

same error with the command and a test folder i made for it. It really looks like the problem comes from my version of windows. I'm on Win10 pro 1809 v17763.1217 x64

Full path is C:\Games\koikatsu. It's the BR install with hf patch installed

from kkmanager.

Looks like it's a locale issue. Different system languages can have different valid values for the /D command from the default Y/N for some reason. You should be able to get it to work by removing the /D Y part of it.

from kkmanager.

Indeed i've just checked the help about the takeown command and the program expects a O instead of a Y for the /D option... That's incredibly stupid from micro$oft... 🤦‍♂️

I guess a fix would be to make kkmanager fetch the locale of win10 and swap Y with whatever letter is used depending of the locale. But is it worth it? 🤷‍♂️

from kkmanager.

Can you try if this build works for you?
KKManager.Core.dll.zip

from kkmanager.

yep, it worked with this file ^^

Except that, once the takeown finishes, although the files are all treated and owned by the right user account, it still says failed:
everyone: the mapping between account names and security IDs hasn't be done. 0 files succesfully treated ; failed on 1 file.
I tried to reboot and try as adviced by the script but it still does it.

from kkmanager.

Can you test if this build still has the mapping error?

KKManager.Core.dll.zip

from kkmanager.

everything is succesful now :)

from kkmanager.