On CVS HEAD current as of 10/19/2005, and link to an archive page yeilds a blank page. Also, I have a custom favicon.ico and the deaul drupal favicon appears instead (in case this aids in debugging).

The "Calendar to browse archives" block appears just fine, but clicking on anything in the calendar just results in the aforementioned blank page.

The blank page consists of:

{html><body></body></html>

(Note: first < changed to { to avoid suspicious content filter.)

No error messages appear.

CommentFileSizeAuthor
#4 legacy_form_get_error.patch7.72 KBgtcaz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented

Sorry for the typos--"and link to an archive page" should be "any link to an archive page," and "deaul drupal favicon" should be "default drupal favicon."

robertgarrigos’s picture

robertgarrigos CreditAttribution: robertgarrigos commented

This is the php error message:

Fatal error: Call to undefined function _form_get_error() in /Library/WebServer/Documents/drupal-head/drupal/includes/legacy.inc on line 422

The '_form_get_error()' function calls in that file, legacy.inc, seems to be wrong. The right one is 'form_get_error()'. Thus by changing all those function calls the problem should be fixed. However, I don't know if this is just a mistake or something made on purpose because of the new form API Drupal developers are talking about these days in Amsterdam.

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented

Good find--I just saw that. I also see the same error on lines 212 and 187 in addition to 422. Will make your change and see what happens.

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented
FileSize
legacy_form_get_error.patch7.72 KB

Here's a patch that replaces all the functions.

robertgarrigos’s picture

robertgarrigos CreditAttribution: robertgarrigos commented
Status: Active » Needs review

Changing status to 'patch (code needs review)' so any of the core developers can check also whether this is something related to the new forms API or not. In any case, they need to know there is a patch to review and commit if it's ok.

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented
Status: Needs review » Active

Intersting that this improved the date widget for the codemonkeyx archive.module but, upon replacing the core archive.module I get the following error:

PHP Fatal error:  Cannot use string offset as an array in includes/form.inc on line 189

This seems to be the same problem as this bug report.

robertgarrigos’s picture

robertgarrigos CreditAttribution: robertgarrigos commented

yup. I just realized it is the same error. We'll need to wait until new form API is applied to the cvs modules.

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented

Robert -- thanks for your help on this. At least we have things narrowed down.

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented

Any updates on this? Would like to use archives and the codemonkeyx archive.module has a one-off for months and days error under HEAD.

gtcaz’s picture

gtcaz CreditAttribution: gtcaz commented
Status: Active » Closed (duplicate)

http://drupal.org/node/35069