[Bumpaneer]

So "invalidcolor" only applies to a button control, and "selected" only applies to a radiobutton, correct?

[Jeroen]

So "invalidcolor" only applies to a button control, and "selected" only applies to a radiobutton, correct?

invalidcolor also applies to edit controls.

[Bumpaneer]

Hmmm...well the code wasn't checking edit controls at all I'll try and get that fixed.

[Bumpaneer]

Here are some code examples :
...
“selected” value
Bello example : https://github.com/Nessus85100/Bello/blo...s.xml#L144
...

Correct me if I'm wrong, but I believe it would be invalid here: https://github.com/Nessus85100/Bello/blo...s.xml#L408

That is a button control and doesn't appear to have a selected attribute according to the wiki. I believe that is what the tool was catching.

[Reynald]

one silly question. Is that with this tool, I can create a file just textures? just the textures file? or it is not as easy and simple? I have doing a MOD Refocus, and I would make the texture of this mod file to have the best speed for small XBMC system as Rasp Pi!

Supplementary question. Is there a tutorial available for this software

Edit: I've been testing and I have invalid xml .. and I do not understand all! Possible to have any advice?

cheers

[nessus]

Correct me if I'm wrong, but I believe it would be invalid here: https://github.com/Nessus85100/Bello/blo...s.xml#L408

That is a button control and doesn't appear to have a selected attribute according to the wiki. I believe that is what the tool was catching.

You are correct. Selected attribute is not valid for simple buttons. I've confuse them with radiobuttons. Sorry for the inconvenience.
So... the only fix that we need is the invalid color attribute.

@Martijn... will send a new PR with fixes for the selected attribute.


Cheers
Nessus

PS. By the way, all these are code corrections, but never were reasons for not accepting to the official repo.

[Bumpaneer]

Here's what I've been working on:

• File names and line numbers clearly displayed
  
• Check files for undefined fonts
  
• Check files for missing images
  
• Fix invalidcolor attribute
  
• Add check for edit controls
  

I've currently got the file names and line numbers working for the Texture and Font checks. I'm working on getting the rest of the code modified and I'll get a new build posted once that happens.

[Hitcher]

Awesome work, many thanks.

[Jeroen]

Here's what I've been working on:

Nice! Thanks for your work on this.

[Bumpaneer]

Found myself short on free time lately, so I wanted to get what I've done so far posted. File names and line numbers should be working for the left 4 buttons indicated in the screenshot below. If you run into any issues please post here so I can fix them before moving on to the right 4 buttons.



Download:
https://dl.dropboxusercontent.com/u/1681...v1.2.3.exe

Thanks for all the testing needed to bring this tool up to date!

[wyrm]

Found myself short on free time lately, so I wanted to get what I've done so far posted. File names and line numbers should be working for the left 4 buttons indicated in the screenshot below. If you run into any issues please post here so I can fix them before moving on to the right 4 buttons.

Download:
https://dl.dropboxusercontent.com/u/1681...v1.2.3.exe

Thanks for all the testing needed to bring this tool up to date!

Bumpaneer,

Sorry mate but Avira virus program flags that file as a virus (HEUR/APC (Cloud) ). You might want to check that you have not picked up a little passenger on your system.

Wyrm (xTV-SAF)

[Sorien]

OT: Maybe instead of creating such small tools like this we could try to create IDE for XBMC plugin/skin development ... JetBrains Idea has very nice plugin system that cover autocomplete/syntax check/code validations/code navigations/custom language injections and so on, we could use Comunity Edition of PyCharm and create XbmcPlugin for it.

[Bumpaneer]

I just changed the packager I was using from .NETZ to ILMerge and re-uploaded the file.

I checked the original and new files with both Avira and Microsoft Security Essentials, latest definitions for both, and it looks clean on my end. Can you please verify you have the latest virus definitions and check the new file?

[wyrm]

I just changed the packager I was using from .NETZ to ILMerge and re-uploaded the file.

I checked the original and new files with both Avira and Microsoft Security Essentials, latest definitions for both, and it looks clean on my end. Can you please verify you have the latest virus definitions and check the new file?

Bumpaneer,

Thanks mate, happy from my end now. Will go have a play and see what it turns up for me.

Wyrm (xTV-SAF)

[Jeroen]

@Bumpaneer:

I am getting a lot of entries like this one:

Code:

Undefined Variable: media - [ViewList.xml : Line 24]

The variable names in question have a ":" in them. Example:

PHP Code:

<variable name="menu:Controls"> 


So it's sort of comparable to this one, I guess the ":" is seen as the end of the var's name.

But it's not a regression or anything, I did't use any variables with a ":" in them at the time of posting the linked post.