Change Image ID Number

[Wayne Workman]

@RobTitian16 Edit the script, at the top is username and password and host. Set those to what is needed. If root works, use root and “password” and set the host to “localhost”

[RobTitian16]

@Wayne-Workman Indeed, I set those as root and “password” with the host being “localhost”. It gave me those errors in the previous screenshot.

[RobTitian16]

@Wayne-Workman

[Wayne Workman]

@RobTitian16 line 17 is actually the if argument. Try to put a space between the [[ and ]] brackets and the inner line between them. Like:
if [[ $snmysqlhost != "" ]]; then

[Tom Elliott]

@RobTitian16 Looks to my like your snmysqlpass and snmysqlhost fields are using missmatching quotes.

The snmysqlpass looks to start with a single quote but end with a double quote.

THe snmysqlhost field looks like it’s much the same.

[Wayne Workman]

@Tom-Elliott Yup. I didn’t look close enough at it lol.

Looks like a simple typo. In the original script I posted, it’s good.

[RobTitian16]

@Wayne-Workman Thanks, although that’s actually a result of me blocking out the password. I can confirm there are double quotes on all 3 lines. I copied and pasted the code below, so it’s all the same as far as I can tell.
I can also confirm there is a space between the [[ and ]] brackets and the inner line between them, as suggested.

[Tom Elliott]

@RobTitian16, @Wayne-Workman is correct.

On all the of the “if lines” you must have a space after [[ and before ]]

[Tom Elliott]

@Wayne-Workman I’d recommend updating this, the if statements are missing spaces.

[Tom Elliott]

@Tom-Elliott Never mind apparently it’s removing excess spaces for some reason.

[RobTitian16]

@Tom-Elliott BAM - that seems to have worked! Thanks!

[Wayne Workman]

@RobTitian16 Try to double quote the variables in there like this:

if [[ "$snmysqlhost" != "" ]]; then

[RobTitian16]

@Wayne-Workman It seemed to work without double quoting the variables - I just had to put spaces, like:

if [[ $snmysqlhost != “” ]] then

[Tom Elliott]

https://github.com/NodeBB/NodeBB/issues/5126

Created issue about the spaces being stripped out.

[Wayne Workman]

@Tom-Elliott said in Change Image ID Number:

@RobTitian16, @Wayne-Workman is correct.

On all the of the “if lines” you must have a space after [[ and before ]]

Tom, you’re correct. And this whole mis-understanding and errors with the script are due to a nodeBB bug. It was stripping out the spaces between the brackets. I’ve checked the script’s original text that I posted by clicking the “edit” button on the post, there are spaces there but the forums isn’t displaying the spaces, so when @RobTitian16 copy/pasted he got a copy without the needed spaces.

[RobTitian16]

@Wayne-Workman Thanks for the assistance with this - it’s much appreciated!
It might be an idea to put this on the wiki once all is done as it would be quite helpful to others, I’m sure

[Wayne Workman]

#wiki worthy

[RobTitian16]

@Wayne-Workman Just an update on this: I tried running the script again after adding an image (the FOG web gui said it was image 31), but the script didn’t appear to run at all - there was no output as before. As a workaround, I had to set the image ID manually, but is this a known issue with the script?

[Tom Elliott]

@RobTitian16 The scripts limitations as they currently stand will only operate if ID 1 is available to be used.

[Wayne Workman]

Tom’s right. I have several ideas on how to make it better but I’ve just not worked on it.