Closed (fixed)
Project:
Content Construction Kit (CCK)
Version:
6.x-1.x-dev
Component:
content.module
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 May 2006 at 13:50 UTC
Updated:
14 Feb 2013 at 11:37 UTC
Jump to comment: Most recent file
Comments
Comment #1
gateone commentedSame here. Error log says:
Comment #2
code rader commentedSimilar errors here. From // $Id: content_admin.inc,v 1.15 2006/05/21 21:37:31 JonBob Exp
Happens when I go to admin/node/types/add
Comment #3
mediafrenzy commentedI'm getting the same errors on a fresh install of latest drupal 4.7 and cvs CCK.
Comment #4
mediafrenzy commented(To clarify - the errors I am seeing are the Scalar Array errors above)
Comment #5
lekei commentedAnybody have the previous build? I picked the WRONG day to update!
Comment #6
mediafrenzy commentedI don't know much about CVS, but is it possible to get the previous version of CCK through the CVS interface perhaps?
Comment #7
pbarnett commentedThe error is caused by content_admin.inc having
when it should be
because stdClass isn't an array.
Comment #8
mfredrickson commentedI can reproduce.
According to http://drupal.org/cvs?commit=32385 types should be arrays, not objects. Attached patch fixes the error.
Comment #9
RayZ commentedComment #10
pbarnett commentedSorry about that; looks like I was fixing the syntax, not the problem. $type=array() certainly fixes it. Thanks, chaps!
Comment #11
EduardoMercovich commentedOK, I have the same problem and the issue is being worked on. I'd like to help and know nothing about programming, but I can test any modifications.
How do I get notified about a new version? Just by visiting this thread or I can subcribe to a feed?
Thanks...
Comment #12
mediafrenzy commentedI'm wondering whether changing to an array has caused issues when trying to use the node_import module with CCK?
As described here:
http://drupal.org/node/65278
Comment #13
RayZ commentedComment #14
Julien PHAM commentedI have the same problem too. Any good fix someone? :p
I'm using CVS version.
The problem about the scalar values.
Comment #15
RayZ commentedDoesn't the patch in #8 fix it for you?
Comment #16
Julien PHAM commentedOh yes sorry :p It works, thanks...
Comment #17
EduardoMercovich commentedAfter searching and reading in Drupal and other places about patching, I tried to apply the patch in #8 (cck.adminerror.patch) with no luck.
Could someone help me by:
a- give the command (in Linux, almost everything about patching in Drupal is for Win), or
b - giving me the patched cck/content_admin.inc?
I know that for devs patching is like breathing, but unfortunately, I am not developer. :-)
Thanks a lot for your help.
Regards...
Comment #18
RayZ commentedIdentical patch submitted by drumm to duplicate issue http://drupal.org/node/65997 . Setting to RTBC.
Comment #19
Torenware commentedI just found a similar problem around line 52 of content_admin.inc (where a call to db_fetch_object needs to get changed to db_fetch_array).
Who do we need to get looking at this one?
v. 1.15 of this file in Drupal CVS needs some QA.
Comment #20
karens commentedHere's a patch that fixes both places, the one fixed in the first patch and the other object reference on line 52.
Comment #21
karens commentedFor those who need it, here is the patched content_admin.inc
Comment #22
karens commentedChanging title to maybe help reduce number of duplicate posts of this problem.
Comment #23
lekei commentedThe way to stop duplicate posts is to fix the bug and re-release the module. People can't believe that a fatal typo-type bug like this isn't fixed immediately, so they report it. And report it. And report it.
If a bug is known and has been fixed but the newest version contains a bug it must be a new bug. A lot of people mistakenly believe that non-programmens can use Drupal.
Don't complain when the fix has been known for almost 2 weeks and the module is still broken. Patches do not count as a fix. They should be considered a testing measure.
Avoid the temptation to patch. Patching modules leads to hacking modules which is the path to the Dark Side.
Remember that many people cannot use patch (or believe that they can't) because they do not have shell access on their host and they are running Windows (I still haven't found a version of patch that works properly on Windows; I finally resorted to the command-line version).
Comment #24
RayZ commentedTrue, but JonBob needs a vacation too (I assume that's why he hasn't been seen for a few weeks?). Anybody else around that can commit a small typo-fix patch like this one?
Comment #25
EduardoMercovich commentedThanks KarenS for your #21 file. :-)
Best regards...
Comment #26
karens commentedThe Drupal commit process was changed recently and only the project maintainers can commit unless they explicitly give permission to someone else to do it, so most likely only JonBob can get a change committed.
lekei, I posted both the patch and the patched file so everyone (programmer or not) can keep using the module until JonBob can get the cvs version updated.
Comment #27
moshe weitzman commentedI emailed JonBob about this and about assigning more committers
Comment #28
jonbob commentedI deeply apologize for the delay. The HEAD patch has been committed.
I cannot reproduce the issue in 4.7 right away, though. Is it happening there still?
Comment #29
(not verified) commentedComment #30
dmitrig01 commentedFound the same thing on line 77. Seting this to active
Comment #31
killes@www.drop.org commentedcan't repro on 4.7.
Comment #32
(not verified) commentedComment #33
abhinya commentedDeclare the array. The error gets removed once you declare the array. In the following example we declare field as an array before declaring the array.