Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: [propel-dev] Re: [propel] Is safe to use propel 1.3?

propel
Discussion topic

Back to topic list

Re: [propel-dev] Re: [propel] Is safe to use propel 1.3?

Reply

Author =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com>
Full name =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com>
Date 2006-11-17 08:47:19 PST
Message That seems a tad too hacky for my taste. I understand that we aren't
using DBAdapters anymore at all, is that correct? Should we maybe re-
introduce some very basic level of abstraction to cater for these
specific situations? For instance, what about this SET NAMES utf8 thing?
Putting stuff like that into the Propel class makes it next to
impossible to customize behavior in userland without patching.

I'll be happy to implement the necessary changes should we decide
it's necessary or even feasible.

Let's hear your thoughts :)


David


Am 17.11.2006 um 17:31 schrieb Soenke Ruempler - NorthClick:

> Soenke Ruempler - NorthClick <> wrote on Friday, November 17, 2006
> 5:22 PM:
>
>> Ron Rademaker <mailto:r.rademak​er@virtualbuilding.n​l> wrote on
>> Friday, November 17, 2006 5:15 PM:
>>
>>> I guess the options from the runtime conf will be set later? (Just
>>> in case anyone ever wants to override the default).
>>
>> Currently not, but it should, yes.
>>
>> I don't know if PDO makes a difference between
>> driver-specific options and
>> global attributes like "PDO::ATTR_EMULATE_PREPARES". Are the
>> constructor-options from $driver_options the same as those
>> are passed to
>> PDO::setAttribute()?
>
> Yes they are. Finally:
>
> Index: Propel.php
> ====================​====================​====================​=======
> --- Propel.php (revision 497)
> +++ Propel.php (working copy)
> @@ -429,8 +429,16 @@
> $user = isset($conparams['user']) ?
> $conparams['user'] : null;
> $password = isset($conparams['password']) ?
> $conparams['password'] : null;
>
> + $driver_options = array();
> +
> + // Toggle PS Emulation for MySQL driver as the
> native server-side
> + // implementation seems to be buggy and slow.
> + // This can be overridden in the connection
> parameters
> + if (strpos($conparams['dsn'], 'mysql') ===
> 0) {
> + $driver_options
> [PDO::ATTR_EMULATE_PREPARES]
> = true;
> + }
> +
> // load any driver options from the INI file
> - $driver_options = array();
> if ( isset($conparams['options']) &&
> is_array($conparams['options']) ) {
> try {
> self::processDriverOptions(
> $conparams['options'], $driver_options );
>
> -soenke
>
> --------------------​--------------------​--------------------​---------
> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org
> For additional commands, e-mail: dev-help at propel dot tigris dot org
>
>

« Previous message in topic | 13 of 16 | Next message in topic »

Messages

Show all messages in topic

Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? Ron Rademaker <r dot rademaker at virtualbuilding dot nl> Ron Rademaker <r dot rademaker at virtualbuilding dot nl> 2006-11-17 06:06:45 PST
     RE: [propel-dev] Re: [propel] Is safe to use propel 1.3? Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> 2006-11-17 06:15:47 PST
         Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? hlellelid Hans Lellelid 2006-11-17 06:18:51 PST
             RE: [propel-dev] Re: [propel] Is safe to use propel 1.3? Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> 2006-11-17 08:00:11 PST
                 Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? hlellelid Hans Lellelid 2006-11-17 08:02:56 PST
                     RE: [propel-dev] Re: [propel] Is safe to use propel 1.3? Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> 2006-11-17 08:12:26 PST
                         Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? Ron Rademaker <r dot rademaker at virtualbuilding dot nl> Ron Rademaker <r dot rademaker at virtualbuilding dot nl> 2006-11-17 08:15:29 PST
                             Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? hlellelid Hans Lellelid 2006-11-17 08:19:08 PST
                             RE: [propel-dev] Re: [propel] Is safe to use propel 1.3? Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> 2006-11-17 08:21:36 PST
                                 Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? hlellelid Hans Lellelid 2006-11-17 08:31:09 PST
                                 Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? Ron Rademaker <r dot rademaker at virtualbuilding dot nl> Ron Rademaker <r dot rademaker at virtualbuilding dot nl> 2006-11-17 08:31:45 PST
                                 RE: [propel-dev] Re: [propel] Is safe to use propel 1.3? Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> Soenke Ruempler - NorthClick <soenke dot ruempler at northclick dot de> 2006-11-17 08:31:55 PST
                                     Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-11-17 08:47:19 PST
                                         Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? Ron Rademaker <r dot rademaker at virtualbuilding dot nl> Ron Rademaker <r dot rademaker at virtualbuilding dot nl> 2006-11-17 08:52:07 PST
                                         Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-11-17 08:52:55 PST
                                             Re: [propel-dev] Re: [propel] Is safe to use propel 1.3? hlellelid Hans Lellelid 2006-11-17 09:59:59 PST
Messages per page: