Closed (fixed)
Project:
e-Commerce
Version:
5.x-4.x-dev
Component:
store
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 Mar 2008 at 15:03 UTC
Updated:
9 May 2008 at 12:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
Phillip Mc commentedJust updating this issue...the same problem exists in the march 24th version of 5.x-4.x-dev.
Comment #2
Phillip Mc commentedJust updating this issue...the same problem exists in the march 25th version of 5.x-4.x-dev.
Comment #3
Phillip Mc commentedjust updating this issue and marking as critical...
the same problem exists in the march 26th version of 5.x-4.x-dev, but this time you can't get past the error..i.e. everytime you try and install any module (even non ec related modules), you get a white screen with the error about store_mail_types().
Comment #4
Phillip Mc commentedBy moving the following lines from store.manage.inc to store.module before installing will get you past the
Call to undefined function store_mail_types() in ec_mail.module on line 458error message.My only guess is that the store.manage.inc file seems to be ignored for some reason during the install process.
You will find the following lines in the store.manage.inc file (starting about line number 695). You need to copy these to the store.module and then delete them from the store.manage.inc before installing the store.module.
No idea if that impacts later...but, at least it gets you past the ecommerce installation process
Comment #5
Phillip Mc commentedHi guys,
just flagging the same error with the march 27th download of 5.x-4.x-dev.
It's a showstopping error - you can't install 5.x-4.x-dev for testing because of it. unless users use the #4 workaround above.
changing the component to be for the store.module (not the ec_mail.module)
cheers
Comment #6
Phillip Mc commentedattaching modified store.module and store.manage.inc files so you can get past the installation errors for the store.module
Note: I've no idea if this impacts on other elements of the ecommerce suite..but, at least it allows you to install for testing. just remove the .txt extension and replace the store.module and store.manage.inc file in your /ecommerce/store/ folder.
Comment #7
open-keywords commentedI used files provided in #6 and seems to work as far as I know
Comment #8
Phillip Mc commentedhi open keywords
can u do me a favour and look to see if your account page has an edit / view tab? When i enable the store.module the edit/view tabs disappear.
thanks.
Comment #9
svilen commentedPhilK
After using the patch my account tabs disappear.
Comment #10
Phillip Mc commentedthanks svilen.
were you able to install the store.module without the patch?
Comment #11
andrejusc commentedNo, taking dev snapshot of March 31 I can't install Store. What do they think? From your thread it seems that it's not fixed for at least 3 weeks by now.
Comment #12
andrejusc commentedActually problem lies in fact that store.manage.inc is not loaded by any code.
What I believe is that somewhere you should have:
include_once drupal_get_path('module', 'store') .'/store.manage.inc';
Comment #13
andrejusc commentedI think you could try this workaround instead. For ec_mail module have one line added:
function ec_mail_implementor_list($types = array(), $module = false) {
include_once drupal_get_path('module', 'store') .'/store.manage.inc';
at least for me it works then without any error/warning. But what I can't understand is to how deal with product attributes. Only via Subproduct? Why it's so user unfriendly?
Comment #14
Phillip Mc commentedThanks andre. good catch with the missing include line.
quick question. when you install the store.module after applying that patch...do you lose the VIEW/ EDIT tabs on your MY ACCOUNT page?
Comment #15
Phillip Mc commentedanswering my own question (#14) ...
yep. the View and Edit tabs for the my account page do disappear after installing the store.module regardless of which patch is used..i.e. #13 or #6.
Comment #16
greg@beargroup.com commentedThanks for the post - just confirming that #13 patch worked for me also to get through the install process.
Comment #17
nicholasthompsonI've just found this - I posted my solution in #251150: Fatal error during instal in 'ec_mail_implementor_list', tries to call 'store_mail_types'
My solution was simply to wrap the offending foreach which called
$functionwith afunction_existscondition.Comment #18
Phillip Mc commentedthanks Nicholas...I think solution #13 above is the correct/better way to solve the problem.
Anyone know how to create a patch for this, to save the ecommerce developers some time?
Comment #19
gordon commentedThis has been fixed now. See http://drupal.org/node/251150
Comment #20
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.