PSP-Archive / ARK-4

Custom Firmware for the PSP and PS Vita.
555 stars 40 forks source link

ARK-4 Plugin Manager showing duplicated entries #350

Closed pebeto closed 2 months ago

pebeto commented 2 months ago

PSP/Vita Model

PSP-1000 (Fat)

ARK Version

ARK-4.20.69

Homebrew/GAME (if applicable)

No response

Plugins ( if applicable )

vsh, ms0:/seplugins/RemoteJoyLite.prx, 1
game, ms0:/seplugins/RemoteJoyLite.prx, 1

Steps to reproduce

  1. Manually add any plugin into your PLUGINS.TXT file under SEPLUGINS
  2. Ensure you have an entry for VSH and GAME
  3. Reset your device, and check Plugins Manager (it can be seen without resetting)

USB Charge

Disabled

Overclock

Disabled

Power Save

Disabled

Balanced Energy Mode

Disabled

Autoboot Launcher

Off

Disable PSP Go Pause

Off

Unlock Extra Memory

Disabled

Memory Stick Speedup

Always

Inferno Cache

LRU

Skip Sony Logos

Off

Hide PIC0 and PIC1

Off

Hide MAC Address

On

Hide DLC

Off

Turn off LEDs

Disabled

Verification of form.

krazynez commented 2 months ago

Its not duplicated one is for VSH and the other is for Game?

pebeto commented 2 months ago

@krazynez

https://github.com/PSP-Archive/ARK-4/assets/28990958/4ece89bd-f8c0-43fb-9ed8-24e88cd55260

krazynez commented 2 months ago

Its not duplicated one is for VSH and the other is for Game?

@krazynez ARK_Plugin_Duplication.mp4

Ahhh I see interesting.

pebeto commented 2 months ago

Found this

char* plugins_path[] = {
    "PLUGINS.TXT",
    "ms0:/SEPLUGINS/PLUGINS.TXT",
    "ef0:/SEPLUGINS/PLUGINS.TXT"
};

The first entry could be the issue (I don't have the SAVEDATA).

krazynez commented 2 months ago

Found this

char* plugins_path[] = {
    "PLUGINS.TXT",
    "ms0:/SEPLUGINS/PLUGINS.TXT",
    "ef0:/SEPLUGINS/PLUGINS.TXT"
};

The first entry could be the issue (I don't have the SAVEDATA).

close extras/modules/xmbctrl/plugins.c:56 had to add some if/else if's to avoid trying to load multiple PLUGIN.TXT, ARK_PATH was saying "Yeah no ARK_01234 so I am going to get SEPLUGINS as my new ARK_PATH" then it would parse the next line (which would do that same thing.

This is going to be in the next rev with another big fix so give me a few.

JoseAaronLopezGarcia commented 2 months ago

Makes sense. Confirm it's fixed and close issue.

JoseAaronLopezGarcia commented 2 months ago

Closing as it should be fixed in latest release.