Bug #119997 for SQL-Translator: S::T::Producer::YAML lose trigger 'scope' property

Thu Jan 26 10:28:19 2017 KES [...] cpan.org - Ticket created

Subject:

S::T::Producer::YAML lose trigger 'scope' property

according to the [DOC](http://search.cpan.org/~ilmari/SQL-Translator/lib/SQL/Translator/Schema/Trigger.pm#scope) trigger has `scope`, but it is [lost when generate YAML file](https://metacpan.org/source/ILMARI/SQL-Translator-0.11021/lib/SQL/Translator/Producer/YAML.pm#L143)

Thu Jan 26 10:32:20 2017 KES [...] cpan.org - Correspondence added

--- a/lib/SQL/Translator/Producer/YAML.pm +++ b/lib/SQL/Translator/Producer/YAML.pm @@ -148,6 +148,7 @@ sub view_trigger { 'fields' => scalar $trigger->fields, 'on_table' => scalar $trigger->on_table, 'action' => scalar $trigger->action, + 'scope' => scalar $trigger->scope, keys %{$trigger->extra} ? ('extra' => { $trigger->extra } ) : (), }; }

Mon Jul 10 07:35:10 2017 ilmari+cpan [...] ilmari.org - Correspondence added

On 2017-01-26 15:28:19, KES wrote: Show quoted text

> according to the [DOC](http://search.cpan.org/~ilmari/SQL- > Translator/lib/SQL/Translator/Schema/Trigger.pm#scope) trigger has > `scope`, but it is [lost when generate YAML > file](https://metacpan.org/source/ILMARI/SQL-Translator- > 0.11021/lib/SQL/Translator/Producer/YAML.pm#L143)

Thanks for the report. This affected the JSON and XML producers and the XML and SQLite parsers as well and is fixed in git by the following two commits: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=dbsrgits/SQL-Translator.git;a=commitdiff;h=c0ec0e22d3f0e3852c00daac5ef5763010b410c3 http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=dbsrgits/SQL-Translator.git;a=commitdiff;h=4384692aca82fb49ad4a49c08d7ddbde85bc4ecb

Mon Jul 10 07:35:13 2017 The RT System itself - Status changed from 'new' to 'open'

Mon Jul 10 07:35:14 2017 ilmari+cpan [...] ilmari.org - Status changed from 'open' to 'patched'