Bug 1211 - new formatting in 2 steps works - not in 1.
: new formatting in 2 steps works - not in 1.
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.0.0
: PC Windows (legacy)
: P5 minor (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-03-27 03:47 UTC by Nalle Johansson
Modified: 2008-08-18 10:54 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nalle Johansson 2005-03-27 03:47:23 UTC
When adding a new formatting and mark the new formatting as the one I want I 
got an errormessage and the new formatting is added but the old one is used. It 
I then klick on the new formatting again - the new one is used. 

Of course this is because os the order the tho thins is donw in the server. And 
this isn�t done so many times so its a minor thing.

Nalle
Comment 1 Blackketter Dean 2005-04-20 14:36:48 UTC
Nalle:  Can you please add specific steps to reproduce the problem?  We don't understand.
Comment 2 Nalle Johansson 2005-05-04 00:03:51 UTC
Start WebbGUI, http://localhost:9000
(Log in)
Click Server Settings
Click Formatting
Scroll down to Title Format
Click in the RadioButton after the last one in use
Enter "File: FILE.EXT" as an example in the textfield to the right of the 
radiobutton.
Click Changebutton

My result this time was:
Title format 12:

File: FILE.EXT

Format will be changed for any clients using this settings New value for 
Current web title format rejected:

         Invalid value specified: 12

The new value "File: FILE.EXT" is in the list if you scroll down but the 
radiobutton has not changed It is still on the old value. If I click in the 
right radiobutton agan and change - everything is fine.



Comment 3 KDF 2005-05-04 12:57:32 UTC
I get it.  I'm not sure how this can be fixed, tho.  The validation for the
radio buttons is built as the page is created.  When you add a new format, the
old validation wont see it the first time.  However, a bit of time pondering may
find a way to allow the 'extra' value.
Comment 4 KDF 2005-05-04 15:55:16 UTC
that needs to happen, is a way to force titleFormat to be processed fully BEFORE
titleFormatWeb is validated so that we can validate titleFormatWeb based on the
new titleFormats. cc'ing moser, got any tips?
Comment 5 Robert Moser II 2005-05-04 22:47:53 UTC
I guess using validateInHash isn't the best choice for the titleformatWeb
preference.  It should probably be changed to validateInteger(0,$arrayMax +
$arrayAddExtra,1,1) with a bit in onChange to drop it down to the $arrayMax if
it is greater than that.

I can make the appropriate changes tomorrow, plus check to make sure there
aren't other prefs in a similar situation.
Comment 6 Robert Moser II 2005-05-05 12:03:37 UTC
Change 3132 for the fix.
Comment 7 Chris Owens 2008-03-11 11:28:02 UTC
This bug was marked resolved in Slimserver 6.1, which is several versions ago.  If you're still seeing this bug, please re-open it.  Thanks!