0TheMeistro0 / wiibrowser

An Internet Browser For Wii
2 stars 0 forks source link

Feature request: Save settings to USB #6

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. change sd:/apps/wiibrowser to usb:/apps/wiibrowser
2. Turn music setting to "off"
3. Restart app

What is the expected output? What do you see instead?
Settings and favorites to save to usb, instead it does not, only allows saving 
to sd card.

What version of the product are you using? On what operating system?
r28

Please provide any additional information below.

Original issue reported on code.google.com by Nathanae...@gmail.com on 20 Jan 2013 at 12:24

GoogleCodeExporter commented 8 years ago
I was taking a look through the source in the settings.cpp file and noticed 
line 127, I'm not a professional programmer by any means so forgive me if I am 
way off the mark, but I was wondering if this line:
        for(int i = SD; i <= USB8; i++)

should be changed to this:
        for(int i == SD; i <= USB8; i++)

Original comment by Nathanae...@gmail.com on 20 Jan 2013 at 9:18

GoogleCodeExporter commented 8 years ago
Thanks for your feedback.
I assume that "change sd:/apps/wiibrowser to usb:/apps/wiibrowser" means you 
changed the "Save Folder" setting from the in-app settings menu:
this setting only affects the path where downloaded pages/files are saved, not 
the configuration path, that is always the app folder (/apps/wiibrowser/) in 
the boot device.

if you try to start the app from usb, with no sd card inserted, settings should 
be saved there.
i'll add an option to define also the configuration path.

About the code line you mention.. well it's fine like it is, because I want to 
assign the value SD to int i, and not to compare the two values :)

Original comment by Gavetheb...@gmail.com on 20 Jan 2013 at 10:01

GoogleCodeExporter commented 8 years ago
Old issue, newer revisions should save settings in the boot device

Original comment by Gavetheb...@gmail.com on 6 Apr 2013 at 1:25