Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#3613 closed defect (fixed)

[PATCH] Trigger points are missing the visibility component

Reported by: AlThePhoenix Owned by: JoshuaJB
Priority: Must Have Milestone: Alpha 19
Component: UI & Simulation Keywords:
Cc: Patch:

Description

ERROR: RelaxNGValidator: Validation error: preview%7Cspecial/trigger_point_A:1: Expecting an element RetainInFog, got nothing

ERROR: RelaxNGValidator: Validation error: preview%7Cspecial/trigger_point_A:1: Invalid sequence in interleave

ERROR: RelaxNGValidator: Validation error: preview%7Cspecial/trigger_point_A:1: Element Visibility failed to validate content

ERROR: RelaxNGValidator: Validation failed for '(null)'

ERROR: Failed to validate entity template 'preview|special/trigger_point_A'

The visibility component wasn't added to the template of trigger points and thus cannot be used in the editor (Survival of the Fittest might be bugged as well, didn't check that yet though).

Attachments (1)

t3613_add_visiblity_component_to_trigger_points_v1.patch (728 bytes) - added by elexis 5 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 5 years ago by elexis

Keywords: patch review added
Summary: Trigger points are missing the visibility component[PATCH] Trigger points are missing the visibility component

Evulant wants it in a19 as it were bad if people couldn't set trigger points on a19 maps.

comment:2 Changed 5 years ago by Josh

Keywords: reviewed added; review removed

Confirmed regression. Trigger placement works in A18 but not in A19. Patch fixes the problem.

comment:3 Changed 5 years ago by JoshuaJB

Owner: set to JoshuaJB
Resolution: fixed
Status: newclosed

In 17270:

Fix #3613 by adding the visibility component to triggers. Patch by elexis.

comment:4 Changed 5 years ago by Josh

Keywords: patch reviewed removed

Thanks for the patch.

Note: See TracTickets for help on using tickets.