There seems to be a problem with adding more than one date/time field to a flexinode, I have attempted to include 3 date fields and only one shows up.

CommentFileSizeAuthor
#2 field_timestamp_1.patch2.16 KBTeddyX

Comments

SpriteGF’s picture

SpriteGF commented
Component: Miscellaneous » Code

I can confirm this with Drupal 4.7.0-beta4 and Flexinode (as of feb 6, 2006). I'd also very much like to get this to work.

To reproduce:
1. Install Drupal.
2. Install Flexinode and import its database definitions.
3. Add a new Flexinode data type (content > add content type)
4. Add a new Date/Time field twice, thrice, or more than thrice.
5. Try creating content for your new Flexinode (create content > [insert flexinode name here])
6. Only one Date/Time field shows up.

TeddyX’s picture

TeddyX commented
Status: Active » Needs review
StatusFileSize
new field_timestamp_1.patch2.16 KB

It seems that attached patch solves the problem. Please review it and commit to CVS.

Bèr Kessels’s picture

Bèr Kessels commented
Version: » 4.6.x-1.x-dev
Component: Code » Field types
Priority: Critical » Normal

Can someone please review?

samc’s picture

samc commented
Version: 4.6.x-1.x-dev » master
Status: Needs review » Reviewed & tested by the community

Ber,

Not sure why the version was changed... I'm seeing this issue on cvs.

Patch is straightforward... replaces hard-coded form name with computed fieldname.

Tests great for me. Thanks!

Bèr Kessels’s picture

Bèr Kessels commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)