hi
report thats the update not work ..d 7 alpha 3

views module
Update #6009

* Failed: Cannot change the definition of field .views_display: field doesn't exist.

regards

CommentFileSizeAuthor
#10 views-765364.patch4.58 KBdawehner
#6 views-7-update.patch3.88 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Closed (duplicate)
eule’s picture

? was heisst hier den man man man ?

du linkst auf die version 2.9 die funktionierte bei meinen drupal 6 installs

eule’s picture

Status: Closed (duplicate) » Closed (won't fix)

Hi ..for me the update is not working

Update #6009

* Failed: Cannot change the definition of field .views_display: field doesn't exist.

Location ./update.php?id=51&op=do
Referrer ./update.php?op=start&id=51
Message Warning: htmlspecialchars() expects parameter 1 to be string, array given in check_plain() (line 1463 of ./includes/bootstrap.inc).
Severity warning

i hope its ok when i reopen the issues

dawehner’s picture

Status: Closed (won't fix) » Active

make it active again, this wasn't a duplicate.

dawehner’s picture

Status: Active » Needs review
FileSize
3.88 KB

This removes the $ret parameter from the functions

eule’s picture

hi
i try today the rel from Views 7.x-3.x-dev (2010-Apr-17)

Notice: Undefined index: db_type in views_update_6009() (line 422 of /modules/views/views.install).
Warning: htmlspecialchars() expects parameter 1 to be string, array given in check_plain() (line 1463 of /includes/bootstrap.inc).

this fix the issues not...do u mean i need the patch on the 7.x-3.x-dev (2010-Apr-17) ?

eule’s picture

i try the update because

--- views.install 8 Apr 2010 21:20:06 -0000 1.50.4.6
+++ views.install 16 Apr 2010 22:36:43 -0000

after the update drupal says me:

The following updates returned messages
views module

in logs i get Notice: Undefined index: db_type in views_update_6009() (line 422 of /modules/views/views.install).

dawehner’s picture

Even if you cannot believe this, thats unlrelated to this issue...

This commit was done to fix a major problem with postqresql, but i cannot figure out yet, how to get the current active db-type

dawehner’s picture

FileSize
4.58 KB

So i found this out, we need someone with pqsql, i cannot test it here.

Letharion’s picture

I have a postgres db that I do testing on, but frankly I don't understand that the problem is.
I'd like to help, but need some info. Is this updating from D6V2 -> D7V3, or something else?

dawehner’s picture

It's "old version of d6v2/3" to d7v3. If you have the current version of views this does not matter.

Letharion’s picture

I see; but since I'm looking for somewhere to start contributing something, I could download an older release, and give it a spin. Still not sure how old a version I would need though.

dawehner’s picture

You would have a views version something around 2.5 and then update to views3 on drupal7 to test this patch.

merlinofchaos’s picture

dereine: Maybe you should just test to make sure this works in mysql and commit it if it does. This issue is pretty old.

dawehner’s picture

Status: Needs review » Needs work

This aren't all changes which are required. There are many more things which have to be ported.

thekevinday’s picture

It looks like the patch here also fixes the db_type issue, so I do not need to re-roll the above patch with my db_driver() patch.

Edit:

I have not seen any problems with the above patch under postgresql.
The problems that I have seen seem to be problems with -beta3 and not with this patch.

imclean’s picture

The patch in #10 applied successfully for me and update.php ran without errors.

add1sun’s picture

Status: Needs work » Reviewed & tested by the community

Encountered the same errors in a D6 > D7 upgrade. Patch in #10 applied with offsets and worked great. This really would be great to roll into the next release since this will stop many people dead in their tracks. I'm with merlinofchaos that you should get at least this patch in and make another issue for further things you want to fix with it.

dawehner’s picture

Status: Reviewed & tested by the community » Needs work

Okay commited this first version.

Set to needs work because some parts of views.install still needs some work.

dawehner’s picture

Status: Needs work » Fixed

Did some additional work, so this should be ready.

It would be really great if someone would like to do some update tests, but this is optional.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.