Avoid division by or modulo of zero in test scenario's maths test (bu…

…g #25214)
wesnoth · Oct 22, 2016 · 50146bc · 50146bc
1 parent 15afec7
commit 50146bc
Showing 1 changed file with 58 additions and 18 deletions.
diff --git a/data/scenario-test.cfg b/data/scenario-test.cfg
@@ -2635,15 +2635,35 @@ For game purposes, the races group into factions; for example, orcs often cooper
                                 variable=divisor
                             [/text_input]
                         [/message]
-                        {VARIABLE total $number}
-                        [set_variable]
-                            name=total
-                            divide=$divisor
-                        [/set_variable]
-                        [message]
-                            speaker=narrator
-                            message="Dividing $number| by $divisor| results in $total|."
-                        [/message]
+                        [if]
+                            [variable]
+                                name=divisor
+                                equals=0
+                            [/variable]
+                            [or]
+                                [variable]
+                                    name=divisor
+                                    equals=
+                                [/variable]
+                            [/or]
+                            [then]
+                                [message]
+                                    speaker=narrator
+                                    message="Cannot divide by zero!"
+                                [/message]
+                            [/then]
+                            [else]
+                                {VARIABLE total $number}
+                                [set_variable]
+                                    name=total
+                                    divide=$divisor
+                                [/set_variable]
+                                [message]
+                                    speaker=narrator
+                                    message="Dividing $number| by $divisor| results in $total|."
+                                [/message]
+                            [/else]
+                        [/if]
                     [/command]
                 [/option]
                 [option]
@@ -2656,15 +2676,35 @@ For game purposes, the races group into factions; for example, orcs often cooper
                                 variable=modulo
                             [/text_input]
                         [/message]
-                        {VARIABLE total $number}
-                        [set_variable]
-                            name=total
-                            modulo=$modulo
-                        [/set_variable]
-                        [message]
-                            speaker=narrator
-                            message="$number| % $modulo| = $total|."
-                        [/message]
+                        [if]
+                            [variable]
+                                name=modulo
+                                equals=0
+                            [/variable]
+                            [or]
+                                [variable]
+                                    name=modulo
+                                    equals=
+                                [/variable]
+                            [/or]
+                            [then]
+                                [message]
+                                    speaker=narrator
+                                    message="Cannot use modulus of zero!"
+                                [/message]
+                            [/then]
+                            [else]
+                                {VARIABLE total $number}
+                                [set_variable]
+                                    name=total
+                                    modulo=$modulo
+                                [/set_variable]
+                                [message]
+                                    speaker=narrator
+                                    message="$number| % $modulo| = $total|."
+                                [/message]
+                            [/else]
+                        [/if]
                     [/command]
                 [/option]
                 [option]