Prefix regen ids with r-

[Qinusty]

Aims to resolve https://github.com/Maradonna90/NewGAN-Manager/issues/186 by prefixing player ids during the writing of portrait records

[trav164]

Just about to do this :)

[Maradonna90]

real question is if we have to change something in the xmlparser.py class when we read in the data from the current config file for the preserve function. Anyone has tested that?

otherwise the proposed change looks good to me.

[filipebarros]

One way would be to add a checkbox with the version of FM the user is playing (FM 24 would prepend the r- and other versions would not

[Qinusty]

real question is if we have to change something in the xmlparser.py class when we read in the data from the current config file for the preserve function. Anyone has tested that?

The parse xml regex will pull the id without the r- prefix and the writer should drop the prefix in on write.

I can't actually successfully build the project as I get a variety of issues building with briefcase on my system. Through windows I get the following when trying briefcase dev.

TypeError: No method matches given arguments for Font..ctor: (<class 'int'>, <class 'System.Drawing.FontStyle'>)

For building for windows, I get a bunch of

TypeError: Command.__init__() got an unexpected keyword argument 'label'

and such when trying to run. I assume there are some python version requirements and/or deps incorrectly versioned?

[Maradonna90]

real question is if we have to change something in the xmlparser.py class when we read in the data from the current config file for the preserve function. Anyone has tested that?

The parse xml regex will pull the id without the r- prefix and the writer should drop the prefix in on write. that would be perfect.

I can't actually successfully build the project as I get a variety of issues building with briefcase on my system. Through windows I get the following when trying briefcase dev.

TypeError: No method matches given arguments for Font..ctor: (<class 'int'>, <class 'System.Drawing.FontStyle'>)

For building for windows, I get a bunch of

TypeError: Command.__init__() got an unexpected keyword argument 'label'

and such when trying to run. I assume there are some python version requirements and/or deps incorrectly versioned?

you need to pull the current master branch into your fork.

and use this version for your local installation

python -m pip install beeware==0.3.0.dev3 toga==0.3.0.dev19 python -m pip install dhooks

but if you're certain this works I can build a version and give it some people for testing.

let me know.

[Maradonna90]

incorporated in different PR