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

http://scriptella.org/dtd/etl.dtd has an error - using <include> in <script>. #29

Open
hey-jude opened this issue Feb 21, 2017 · 1 comment

Comments

@hey-jude
Copy link

hey-jude commented Feb 21, 2017

I think dtd has an error.

I wrote xml and xml validator reports "include" in "script" is not allowed,
but it can be executed.

I guess that the definition of "include" in "script" in "query" is omitted.

  • the definition of "include" in "script" only has no dtd error.

It's my sample xml.

<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE etl SYSTEM "http://scriptella.org/dtd/etl.dtd">
<etl>
    <description>test skeleton</description>
    <properties>
        <include href="etl.properties"/> <!--Load from external properties file-->
    </properties>
    <!-- Connection declarations -->
    <connection id="con1" driver="$driver" url="$source.url" user="$source.user" password="$source.password" classpath="$classpath"/>
    <connection id="con2" driver="$driver" url="$target.url" user="$target.user" password="$target.password" classpath="$classpath"/>

    <query connection-id="con1">
        <include href="$query_sql" />
        <script connection-id="con2">
            <include href="$insert_sql" /> <!-- error reporting point -->
        </script>
    </query>
</etl>
@ejboy
Copy link
Member

ejboy commented Dec 14, 2019

Hmm. In Intellij XML editor is indeed reported as error inside <script>. But the definition in DTD is the same:

<!ELEMENT script (#PCDATA | include | dialect | onerror)*>
<!ELEMENT query (#PCDATA | include | dialect | query | script)*>

So I'm not sure where does the error come from. Let me know if you can point the issue in dtd file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants