Hello,
I am running Drupal 6.16 on a WAMP server with Apache 2.2.11, MySQL 5.1.36 and php 5.3.0.
I used to have php 5.2.11 up until the Drupal 6.14 release, time at which I upgraded to php 5.3.0 which is the php version that comes with WAMP 2.0i (see www.wampserver.com and www.drupal.org/requirements)
Node Import does not work with php 5.3.0. I get the following warning message at step 7/8:
warning: Parameter 2 to drupal_retrieve_form() expected to be a reference, value given in C:\wamp\www\sites\mif\modules\node_import\node_import.inc on line 1842.
If I ignore it and proceed with the import in step 8/8, it says that all rows were imported but they actually were not. The content is not visible when looking at the content page (admin/contnet/node) nor is it visible when looking at the node table with phpMyAdmin
When I go back to php 5.2.11, it does actually import all the rows very well.
Is there something (e.g. a php extension that Node Import needs) that may not be set up properly?
Thanks a lot for your help! :)
Christian
Comment | File | Size | Author |
---|---|---|---|
#14 | node_import_php5.3_fix.patch | 746 bytes | tylor |
Comments
Comment #1
Robrecht Jacques CreditAttribution: Robrecht Jacques commentedCould you test replacing:
with
in
sites/all/modules/node_import/node_import.inc
? It should be around line 1842.Thanks.
(EDIT: forgot the "must_validate" line)
Comment #2
ch_masson CreditAttribution: ch_masson commentedRobrecht,
Sorry for the late reply. Yes, that fixed the issue.
Thanks,
Christian
Comment #3
ch_masson CreditAttribution: ch_masson commentedComment #4
ch_masson CreditAttribution: ch_masson commentedComment #5
Robrecht Jacques CreditAttribution: Robrecht Jacques commentedNeeds to be committed.
Comment #6
j0rd CreditAttribution: j0rd commentedNeeds to be committed. Subscribing. Had to find and apply this patch twice manually already.
Comment #7
j0rd CreditAttribution: j0rd commentedtagging as 5.3 bug.
Comment #8
Coupon Code Swap CreditAttribution: Coupon Code Swap commentedThanks
http://drupal.org/node/763036#comment-2878702
works for me with PHP 5.3.3
Comment #9
asad.hasan CreditAttribution: asad.hasan commentedWorks like a beauty!
Comment #10
imclean CreditAttribution: imclean commented#1 works for me. Still not committed so I wouldn't call it "fixed" just yet.
Comment #11
grouchyladybug CreditAttribution: grouchyladybug commented#1 works for me on PHP 5.3.1...glad to find this after a day of banging my head on the keyboard trying to figure out where my supposedly imported nodes were!
Comment #12
Sutharsan CreditAttribution: Sutharsan commentedWorks as advertised. Marking as critical since PHP 5.3 is used more and more.
Comment #13
fonant CreditAttribution: fonant commentedOne-line fix in #1 works for me with PHP 5.3. Should be safe to commit.
Comment #14
tylor CreditAttribution: tylor commentedTested #1 as working properly, created a patch file so I can use this in a drush make file until it gets committed. Thanks for the fix!
Comment #15
PetarB CreditAttribution: PetarB commentedI ran into this issue. After following Robrecht's advice in #1 the patch worked. Thank you.
Comment #16
Renee S CreditAttribution: Renee S commentedWorked for me. Commit plz?? :)
Comment #17
itserich CreditAttribution: itserich commentedI have a problem where the imports appear fine and confirm states it worked but no nodes are created, much like the original issue.
However, I am using PHP 5.2.14
Can anyone state whether that would be the cause of my failure?
I desperately need to import a bunch of nodes and actually importing them is the last step.
Thank you.
Comment #18
basvredelingI checked the latest dev but it's still not committed. Please add this fix.
Comment #19
joe3zzz CreditAttribution: joe3zzz commentedworked for php 5.3.4 - thanks!
Comment #20
kclarkson CreditAttribution: kclarkson commentedHas this fix been committed?
I am not a big fan of patches.
Thanks,
K
Comment #21
zahidansari CreditAttribution: zahidansari commentedTested with php 5.3.2 successfully . One issue though on step 7
Use Case: I have validity date in content type which I forgot to give in step 4 . When I reached to step 7 it showed me the whole code
Record 1:
* Validity Date field is required.
* Validity Date field is required.
after which it shows this code ::
Comment #22
Robrecht Jacques CreditAttribution: Robrecht Jacques commentedCommitted.
Comment #25
YesCT CreditAttribution: YesCT commented