Skip Menu |
 

This queue is for tickets about the SQL-Translator CPAN distribution.

Report information
The Basics
Id: 119997
Status: patched
Priority: 0/
Queue: SQL-Translator

People
Owner: Nobody in particular
Requestors: KES [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: S::T::Producer::YAML lose trigger 'scope' property
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.18-25070-1485444499-418.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 274
Download (untitled) / with headers
text/plain 274b
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-25070-1485444499-418.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-25070-1485444499-418.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-4772-1485444740-272.119997-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 446
Download (untitled) / with headers
text/plain 446b
--- 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 } ) : (), }; }
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-25070-1485444499-418.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-25070-1485444499-418.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-31284-1499686510-161.119997-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 738
Download (untitled) / with headers
text/plain 738b
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


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.