No subject

Sun Nov 20 20:48:46 GMT 2022

```
Actually, it's not an issue for me anymore because I have switched to 
generating my schema files from the database to generating the database 
from the schema files.  Though I don't know if it may be an issue for 
others.

On 1/21/2011 12:19 PM, Rafael Kitover via RT wrote:
> <URL: https://rt.cpan.org/Ticket/Display.html?id=59640>
>
> Hi,
>
> is this still an issue for you? The lack of size in bytes for integers
> etc. in the generated Result classes.
>
> I see you are asking for some sort of configuration option, so I'm not
> closing the ticket yet.
>
> What would this option do? Just set size to size in bytes rather than
> DDL size?

```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/dbsrgits/dbix-class-schema-loader/issues/47
You are receiving this because you are subscribed to this thread.

Message ID: <dbsrgits/dbix-class-schema-loader/issues/47 at github.com>
----==_mimepart_637a934f5469c_752fc67019295b9
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

Migrated from <a href="https://rt.cpan.org/Ticket/Display.html?id=59640" rel="nofollow">rt.cpan.org#59640</a> (status was 'open')
Requestors:
<ul dir="auto">
<li><a href="mailto:maz at mlx.net">maz at mlx.net</a></li>
</ul>
From <a href="mailto:maz at mlx.net">maz at mlx.net</a> on 2010-07-23 06:38:09 
:
<pre class="notranslate"><code class="notranslate">Versions of DBIx::Class::Schema::Loader prior to Version 0.06 always 
included the "size" definition for mysql databases. Version 0.06+ no 
longer includes "size" definitions for some fields such as integers. 
This breaks any script that depends on the "size" definition. Could we 
either put the size definition back in or make it a configurable 
option? This breaks all my validation routines which depend on the 
"size" definition.

Sincerely,

John Maslanik

</code></pre>
<p dir="auto">From <a href="mailto:rkitover at cpan.org">rkitover at cpan.org</a> on 2010-07-27 21:36:50<br>
:</p>
<pre class="notranslate"><code class="notranslate">Hi John,

The "size" means the size of the type when declared in DDL, for example
for VARCHAR(30) the "size" is 30.

INTEGER and such do not have a size for DDL purposes, this is why it is
omitted.

Previous versions of ::Schema::Loader just set size to the size in bytes
are reported by DBI, this was wrong and broke deploying the generated
schemas with $schema-&gt;deploy using SQL::Translator.

Current versions of ::Schema::Loader are much more SQL::Translator
friendly and have accurate type information for the purposes of
deployment and cross-deployment.

If you need the old behavior, make a subclass of
::Schema::Loader::DBI::mysql that overrides _columns_info_for and pass
that in as the "loader_class".

On Fri, Jul 23, 2010 at 02:38:10AM -0400, John Maslanik via RT wrote:
&gt; Versions of DBIx::Class::Schema::Loader prior to Version 0.06 always 
&gt; included the "size" definition for mysql databases. Version 0.06+ no 
&gt; longer includes "size" definitions for some fields such as integers. 
&gt; This breaks any script that depends on the "size" definition. Could we 
&gt; either put the size definition back in or make it a configurable 
&gt; option? This breaks all my validation routines which depend on the 
&gt; "size" definition.
&gt; 
&gt; 
&gt; Sincerely,
&gt; 
&gt; John Maslanik
&gt; 
&gt; 
</code></pre>
From <a href="mailto:rkitover at cpan.org">rkitover at cpan.org</a> on 2011-01-21 04:19:13 
:
<pre class="notranslate"><code class="notranslate">Hi,

is this still an issue for you? The lack of size in bytes for integers
etc. in the generated Result classes.

I see you are asking for some sort of configuration option, so I'm not
closing the ticket yet.

What would this option do? Just set size to size in bytes rather than
DDL size?
</code></pre>
From <a href="mailto:maz at mlx.net">maz at mlx.net</a> on 2011-01-21 05:42:59 
:
<pre class="notranslate"><code class="notranslate">Actually, it's not an issue for me anymore because I have switched to 
generating my schema files from the database to generating the database 
from the schema files. Though I don't know if it may be an issue for 
others.

On 1/21/2011 12:19 PM, Rafael Kitover via RT wrote:
&gt; &lt;URL: https://rt.cpan.org/Ticket/Display.html?id=59640&gt;
&gt;
&gt; Hi,
&gt;
&gt; is this still an issue for you? The lack of size in bytes for integers
&gt; etc. in the generated Result classes.
&gt;
&gt; I see you are asking for some sort of configuration option, so I'm not
&gt; closing the ticket yet.
&gt;
&gt; What would this option do? Just set size to size in bytes rather than
&gt; DDL size?

</code></pre>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />Reply to this email directly, <a href="https://github.com/dbsrgits/dbix-class-schema-loader/issues/47">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AACJ4ARBBQML4BKF47R6DSTWJKFM7ANCNFSM6AAAAAASGAV4ZE">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AACJ4AWRKKEGRY75WJSPCZTWJKFM7A5CNFSM6AAAAAASGAV4ZGWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHFNWFR7Q.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span>&lt;dbsrgits/dbix-class-schema-loader/issues/47</span><span>@</span><span>github</span><span>.</span><span>com&gt;</span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/dbsrgits/dbix-class-schema-loader/issues/47",
"url": "https://github.com/dbsrgits/dbix-class-schema-loader/issues/47",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_637a934f5469c_752fc67019295b9--