No upgrade path from Drupal 5 [#339335]

Comment #1

litwol CreditAttribution: litwol commented 26 November 2008 at 17:38

Priority:	Critical	» Normal
Status:	Active	» Closed (won't fix)

Yes its by design. If some one is interested in this enough to create upgrade path i will gladly commit it.

Log in or register to post comments

Comment #2

beginner CreditAttribution: beginner commented 27 November 2008 at 01:26

Priority:	Normal	» Critical
Status:	Closed (won't fix)	» Active

Nothing has been fixed. leave the issues open at least for the sake of the other users so that they don't suffer the problems I encountered because of those bugs.
If you prematurely close the issues, they won't be visible and it's neither me nor you who will suffer the consequences of not knowing about this issue.

Log in or register to post comments

Comment #3

deekayen CreditAttribution: deekayen commented 4 December 2008 at 20:51

Status:

Active

» Needs review

function mail_logger_update_6000() {
  $ret = array();
  db_add_column($ret, 'mail_logger', 'language', 'varchar', array(
    'description'   => t('Language code'),
    'length'        => 20,
    'not null'      => TRUE,
    // default is required for PostreSQL according to http://api.drupal.org/api/function/db_add_column/6
    'default'       => "''"
  ));
  db_add_index($ret, 'mail_logger', 'language', 'language');
}

Log in or register to post comments

Comment #4

deekayen CreditAttribution: deekayen commented 24 February 2009 at 18:55

Status:

Needs review

» Needs work

buggy... making a revision right now

Log in or register to post comments

Comment #5

deekayen CreditAttribution: deekayen commented 24 February 2009 at 19:07

Status:

Needs work

» Needs review

function mail_logger_update_6000() {
  $ret = array();
  db_add_field($ret, 'mail_logger', 'language', array(
      'description'   => t('Language code'),
      'type'          => 'varchar',
      'length'        => 20,
      'not null'      => TRUE,
    )
  );
  db_add_index($ret, 'mail_logger', '`language`', array('`language`'));
  return $ret;
}

Log in or register to post comments

Comment #6

litwol CreditAttribution: litwol commented 5 February 2010 at 20:39

Can some one review this to make sure it runs ? I dont use this module on any of my sites so i have no data to test it with.

Log in or register to post comments

Comment #7

deekayen CreditAttribution: deekayen commented 13 September 2010 at 02:17

Status:

Needs review

» Fixed

I wrapped it in a db_column_exists check since it's been so long, that way it'll be ignored by people who started with this module installing as Drupal 6.