Development

#3364 ([PATCH] enable compat_assoc_lower and compat_rtrim_string in sfPropelDatabase)

You must first sign up to be able to contribute.

Ticket #3364 (closed enhancement: fixed)

Opened 8 months ago

Last modified 7 months ago

[PATCH] enable compat_assoc_lower and compat_rtrim_string in sfPropelDatabase

Reported by: Rob.Sworder Assigned to: fabien
Priority: minor Milestone: 1.0.14
Component: model Version: 1.0.12
Keywords: propel sfPropelDatabase mssql compat_rtrim_string Cc:
Qualification: Unreviewed

Description

Although compat_assoc_lower and compat_rtrim_string are enabled in sfCreoleDatabase, and are read in from databases.yml, sfPropelDatabase filters them out. MSSQL seems to suffer from issues such as empty string being converted to ' ', so the compat_rtrim_string flag is desirable to be applied in this case.

Attachments

patch.txt (491 bytes) - added by Rob.Sworder on 04/18/08 12:39:35.
patch containing the additional fields
diff.txt (1.0 kB) - added by Rob.Sworder on 04/30/08 17:56:50.

Change History

04/18/08 12:39:35 changed by Rob.Sworder

  • attachment patch.txt added.

patch containing the additional fields

04/18/08 16:37:59 changed by fabien

  • milestone set to 1.0.14.

04/30/08 17:50:19 changed by Rob.Sworder

Forgot to say, patch is against addon/propel/database/sfPropelDatabase.class.php

04/30/08 17:56:50 changed by Rob.Sworder

  • attachment diff.txt added.

04/30/08 18:42:29 changed by FabianLange

  • status changed from new to closed.
  • resolution set to fixed.

(In [8699]) added compat options to propel databse (sfPropelDatabase.class.php) (fixes #3364)