Rails Ruby on Rails → 2.x

Comments and changes to this ticket

• You flagged this item as spam.
  

  Nuno Job October 2nd, 2008 @ 04:23 PM

  I have been using Ruby on Rails/DB2/XML for quite some time. This would turn things easier for other developers like myself. Nice idea!

• You flagged this item as spam.
  

  Philip Nelson October 2nd, 2008 @ 07:59 PM

  Something I've been wanting to happen for so long !!!

  I can't imagine how many times I've patched the Rails code manually in the last two years or so, since I started doing Rails development with DB2.

  Phil Nelson ScotDB Limited

• You flagged this item as spam.
  

  Leonardo Borges October 2nd, 2008 @ 08:16 PM

  That really sounds nice!

  I found myself patching rails code as well and this would be a huge step forward.

  At our company we still go a bit further and use DB2's native XML support with rails, so this is definitely something we'd like to see.

• 

  Pratik October 3rd, 2008 @ 12:50 AM

  • Assigned user set to “Pratik”
• 

  Pratik October 3rd, 2008 @ 01:01 AM

  Looks nice. Some suggestions :

  • Please use 'development', 'test', '_production' postfixes.
  • Instead of commented values for schema, host, port, account etc. why not fill sensible defaults ? Not being an IBM DB user, I don't know what they might be.
  • I'd skip those tests

  Thanks.

• You flagged this item as spam.
  

  Antonio Cangiano October 3rd, 2008 @ 02:42 AM

  Pratik, thank you for reviewing the patch.

  I proceeded with reviewing the patch developed by the API development team, and changed a few things, also incorporating some of your suggestions.

  A couple of important considerations are in order though:

  • The development, test and _production postfixes are not acceptable for DB2. This is due to the fact that database names are currently limited to 8 characters. This is why the resulting database names are derived from the application name (truncated to 4 characters) and contracted postfixes;

  • I changed the parameters for which default values exist, to their defaults. That said, some of the optional parameters are entirely arbitrary and have no defaults, as such I placed obvious placeholders. The password field is empty for the other adapters, so I left it blank as well.

  I think the resulting config/database.yml file looks good, all things considered and it's ready to be introduced into the core.

  Thank you, Antonio

• You flagged this item as spam.
  

  Antonio Cangiano October 3rd, 2008 @ 02:52 AM

  Please find attached the new patch.

  Thanks.

  • add_ibm_db_to_database_option.diff 3.2 KB
• 

  Repository October 4th, 2008 @ 01:56 PM

  • State changed from “new” to “resolved”

  (from [f0e90740d0ca35b3fae5cf367334929534c35555]) Add IMB DB support to Rails application generator. [#1136 state:resolved]

  Usage : rails app_name -d ibm_db

  Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...