Change Image ID Number

[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.

[RobTitian16]

@Tom-Elliott Ah that makes sense. Thanks for clarifying!