Error in transformation not shown in the client

bugs
1

A transformation (complex data rule) added a float value 0.0 to an integer field. The error was not shown in the client or in the log. The result of a sequence of previously run rules was ignored without any explanation why it happened.

2
3
5

The error is silently caught by .net framework. Now it will be logged at least.

We should think about even better solution that would produce an error message when a data rule fails.

6

We should save the exceptions thrown in the RowChanged handler and throw them later if we cannot chnage the behavior of the DataTable. This should be added to master only. The logging is sufficient for 2021.1.

7

I am almost certain it worked before. @zcapkova can you please make a small test in the old version and see if it ends up with an error message? The test would be to create a complex data rule which would throw an error, like this:

<xsl:message terminate="yes">
Test error
</xsl:message>
8

Complex data rule written like this threw no error, either in the client or in the log.

<?xml version="1.0" encoding="UTF-8"?>
<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0"
	xmlns:AS="http://schema.advantages.cz/AsapFunctions"
	xmlns:date="http://exslt.org/dates-and-times" exclude-result-prefixes="AS date">

	<xsl:template match="ROOT">		
		<ROOT>
			<xsl:message terminate="yes">test error</xsl:message>
		</ROOT>
	</xsl:template>

</xsl:stylesheet>
9

So it does nothing in .net 4 (Flash) backend?

10

That was a misunderstanding of what “old version” is - I tried it with the uncorrected rule in html, not in flash.

BTW - it showed the error message when I was saving the rule in Architect.

11

Also nothing in Flash. But that is only for the rule, in transformations xsl:message works fine.

12

@tvavrda we established that the problem with running rules when they are run as a side effect by another rule was present in flash as well. Can we call the solution implemented here sufficient and close this issue?

13

I would call it a “known issue” that should be fixed one day though.

14

This topic was automatically closed 2 days after the last reply. New replies are no longer allowed.