In the function ninesixty_preprocess_page(), I have seen D6-format calls for the function theme().

The small attached patch rewrite them in D7-format.

CommentFileSizeAuthor
ninesixty_preprocess_page_101028.patch1.91 KBjpsoto
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

trogie’s picture

trogie CreditAttribution: trogie commented
Status: Needs review » Reviewed & tested by the community

If you allow me to call myself 'the community' :-)

skwashd’s picture

skwashd CreditAttribution: skwashd commented

I found quite a few more problems when trying to use ninesixty with D7b2. I have posted a patch (#960968: Update code for Drupal 7 RC3) which deals with most of them. As my patch deals with a lot more of D7 issues, I'd suggest this one gets duped against the issue I opened. Reviews welcomed.

jpsoto’s picture

jpsoto CreditAttribution: jpsoto commented

@skwashd I agree, please feel free to change status to "closed (duplicated)".

However, let me to point up patches have to be relative to the CVS repository. Your contribution could not be comitted because it has been generated relative to a tarball.

I think you should consider to work against CVS.

Anyway, ... thanks a lot for your contribution.

skwashd’s picture

skwashd CreditAttribution: skwashd commented
Status: Reviewed & tested by the community » Closed (duplicate)

@jpsoto the diff was taken against CVS, but I'd copied the fixed info file from my initial workspace which was based off the latest -dev tarball.

Closing - DUPE