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

There is a problem with the script "Body volcanism report" at line 37. expected expression, found #2618

Closed
johnny2trayz opened this issue Jun 13, 2024 · 4 comments
Assignees
Labels
8. annoyance The behaviour is as specified, but has been found to be annoying in practice by real-world users.

Comments

@johnny2trayz
Copy link

No clue why she's saying that when I'm scanning planets.
speechresponder.txt

@johnny2trayz
Copy link
Author

Also just got this:

There is a problem with the script "Entered normal space" at line 34. expected end of file, found closing curly bracket

@Darkcyde13
Copy link

Darkcyde13 commented Jun 13, 2024

Is this in the new 4.0.4-b1 beta? If so...

Looking at that error, I think it's probably some code in your script that uses an escape sequence. I'm guessing there is \\ somewhere to escape the character that follows, usually a " or ' quote mark that's used inside another " or ' pair for a string. This is what's happening in the new Destination arrived script too.

If this is the case, you will need to remove one \ and probably update the type of quote mark used. So for example, if you have this:
"Some text here '{OneOf(\\'more text\\', \\'other text\\')}' last text"
You will need to change it to this:
"Some text here '{OneOf(\"more text\", \"other text\")}' last text"

However, if you have something like this:
"You {OneOf('have no', 'don\\'t have any')} mission route"
change it to:
'You {OneOf("have no", "don\'t have any")} mission route'

The double slash \\ was working in version 4.0.3, but apparently it should not have been, and it's been fixed for 4.0.4-b1. And, as far as I can tell, one type of quote mark must be followed by the other type for it to work now, as demonstrated in the above examples. Also see here: #2614

The Entered normal space error has been reported on the forums already, but thanks for reporting it here too. 🙂

@johnny2trayz
Copy link
Author

I don't remember really modifying anything in EDDI other than setting an IVONA voice. Is there a way to reset everything? For now I just made a copy of the main eddi and disabled those triggers.

@Tkael Tkael self-assigned this Jun 14, 2024
@Tkael Tkael added the 8. annoyance The behaviour is as specified, but has been found to be annoying in practice by real-world users. label Jun 14, 2024
@Tkael
Copy link
Member

Tkael commented Jun 14, 2024

Thank you @johnny2trayz , this should be fixed in the next release.

@Tkael Tkael closed this as completed Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8. annoyance The behaviour is as specified, but has been found to be annoying in practice by real-world users.
Projects
None yet
Development

No branches or pull requests

3 participants