curiousdannii-testing / inform7-imported-bugs

0 stars 0 forks source link

[I7-1714] [Mantis 1750] Impossible to modify 2 responses in Locksmith #370

Closed curiousdannii-testing closed 2 years ago

curiousdannii-testing commented 2 years ago

Reported by : Natrium729

Description :

Two responses in Locksmith are impossible to modify. These are:

When you write:
must have accessible the noun rule response (A) is "[Tu] ne [peux] rien faire [si tu] [ne tiens pas] [the noun]."

Inform thinks the verb is "have" and not "is" and then complains.

Steps to reproduce :

"Locksmith Responses"

Include Locksmith by Emily Short.

must have accessible the noun rule response (A) is "[Tu] ne [peux] rien faire [si tu] [ne tiens pas] [the noun].".
must have accessible the second noun rule response (A) is "[Tu] ne [peux] rien faire [si tu] [ne tiens pas] [the second noun].".

There is a room.

Additional information :

A simple solution would be to change the rules' names.

It would be better yet to be able to force Inform to read the correct verb in an assertion with parenthesis, like:
(must have accessible the noun rule response (A)) is ("[Tu] ne [peux] rien faire [si tu] [ne tiens pas] [the noun].")

imported from: [Mantis 1750] Impossible to modify 2 responses in Locksmith
  • status: Closed
  • resolution: Resolved
  • resolved: 2022-04-15T08:45:17+10:00
  • imported: 2022/01/10
curiousdannii-testing commented 2 years ago

61eedb62875fc10070240916:

Fixed via this commit:
https://github.com/ganelson/inform/commit/0f3fb2e2d270eaee81b80284490f76ac4c201051

Comment by Graham Nelson: After some consideration, I agree with Natrium729 that the best thing is to give these two rules more sensible names, since their names are currently very odd in English anyway.

must have accessible the noun rule -> noun must be accessible rule
must have accessible the second noun rule -> second noun must be accessible rule