Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Server GUI directory should default to "None" (not registered) #2441

Closed
softins opened this issue Feb 28, 2022 · 2 comments · Fixed by #2442
Closed

Server GUI directory should default to "None" (not registered) #2441

softins opened this issue Feb 28, 2022 · 2 comments · Fixed by #2442
Milestone

Comments

@softins
Copy link
Member

softins commented Feb 28, 2022

I have just tried this code for the first time (in fact it was the code for #2427 I tried, which incorporates these changes), and compared with 3.8.2, using default empty ini files to ensure all defaults were used:

/usr/local/bin/Jamulus -s -i /tmp/x.ini
./Jamulus -s -p 22125 -i /tmp/y.ini

The 3.8.2 defaulted to "Not registered", with the "make my server public" checkbox unticked, which is good.

The newer version defaulted to registering with Any Genre 1, which I think is not good. The directory drop-down should default to "None", if not set in the command line or ini file, so that a new server does not register with a directory until the user asks it to.

Originally posted by @softins in #2199 (comment)

softins added a commit to softins/jamulus that referenced this issue Feb 28, 2022
Fixes jamulussoftware#2441. If the directory is not set in the ini file or the command line,
the server should not automatically register to All Genres 1. It should
default to not registering until the user actually chooses a directory.
@pljones
Copy link
Collaborator

pljones commented Feb 28, 2022

The old client did this (watch the console output):

  • On start up, default to registered with Any Genre 1
  • Read the genre from the inifile, default to Any Genre 1
  • Read the registration state from the inifile, default to unregistered
  • Update the registration state

The new client says:

  • Read the custom directory from the inifile, default to not set
  • Read the genre from the inifile, default to Any Genre 1
  • Set the registration state

So defaulting to Any Genre 1 hasn't changed but the effect has.

This is why the checkbox was a "hack", in my view - it causes a problem as you don't actually know where the user wanted to register and, when you check the box, you're forced to register with Any Genre 1.

@softins
Copy link
Member Author

softins commented Feb 28, 2022

Yes, not enabling the form items to select the directory until you had enabled it was a design error in the versions up to and including 3.8.2. Now that the enabling of registration has been subsumed into the directory selection, this gives us the opportunity to fix that error and prevent the server registering anywhere until the user has said where. That's what this PR does.

@pljones pljones added this to the Release 3.9.0 milestone Mar 4, 2022
pgScorpio pushed a commit to pgScorpio/jamulus that referenced this issue Mar 28, 2022
Fixes jamulussoftware#2441. If the directory is not set in the ini file or the command line,
the server should not automatically register to All Genres 1. It should
default to not registering until the user actually chooses a directory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants