Hi,

Thanks for the wonderful module. One bug I'm running into is when using the facebook_status module (which works like twitter statuses). Every time a user inputs a status update they get the following:

 * warning: array_shift() [function.array-shift]: The argument should be an array in /var/www/lc3/sites/all/modules/facebook_status/facebook_status.module on line 1012.
    * warning: call_user_func_array() [function.call-user-func-array]: First argument is expected to be a valid callback, '' was given in /var/www/lc3/includes/form.inc on line 366.

IceCreamYou (maintainer of FB_Status) has suggested that since the site in question only experiences said issue with Mobile Tools turned on that MT is conflicting with this line in the FB_Status module:

<?php
  $args = $form['#parameters'];
  $form_id = array_shift($args);
?>

He also suggests that this line should work and shouldn't be in conflict with Mobile Tools. Check the following link for more details: http://drupal.org/node/517296.

Comments

twom’s picture

twom commented
Assigned: Unassigned » twom

Thank you for reporting this issue. I'll have a look at the problem

twom’s picture

twom commented

Hey,

I am currenlty not able to reproduce the error. Can you check if you have the 1.7 version, because in previous versions I had a problem with the opening php tag that caused this kind of problems...

In meantime I'll do some more testing.

pribeh’s picture

pribeh commented

Thanks for responding so quickly twom. I've tested with both the 1.7 and the current dev version.

pribeh’s picture

pribeh commented

Hi, I'm quite sorry as I've forgotten to fill in some more details. I'm using facebook_status with an attached view that get's recalled with some javascript. Documentation of how I'm doing this can be found here (under "JavaScript View Updates"): http://drupal.org/node/435676#auto

pribeh’s picture

pribeh commented

Hi twom, I just basically did some more bug testing with this issue and found out a little more from the maintainer of FBSS. I setup FBSS status form to refresh the page upon submission of a status. I'm using the latest devs from both Mobile Tools and FBSS. Now what I get is a "NaN" error on the status form which IceCreamYou explains in #13 of this thread: http://drupal.org/node/517296#comment-2158906 . Is there any possibility that Mobile Tools might possibly be trying to pull in more than one instance of things like the FBSS status form? Thanks for your help.

twom’s picture

twom commented

Hi,

I have not yet found the time to really fix this issue. I currently don't have a clue where things could go wrong....
If you have more info... Please post! In meantime I have uploaded a new dev version that is more stable.

I have done a major code clean-up and refreshing of the user interface.... It would be great if you could test it and see if the error persists...

pribeh’s picture

pribeh commented

Great stuff twom. I'll try it out and report back pronto.

twom’s picture

twom commented
Status: Active » Needs review

I hope that this thread can be activated again. I currently fixed a critical error that cause erros when retrieving a json response (using ajax).
I hope this fix will avoid the problem you have here!

Someone can verify?

pribeh’s picture

pribeh commented

Cool. Thanks for actually taking the time to update this. I'll test this asap. I've actually been waiting on a potential fix for this to do the mobile theme for my site.

Mauro Colella’s picture

Mauro Colella commented

Hello everybody. I encountered the same issue, and am able to provide details since I did investigate a bit. It seems the conflict occurs in hook_forms. It also confuses other Drupal features such as the "uninstall modules" - I had to disable Mobile Tools because I couldn't manually uninstall anymore :/

I am on : Drupal 6.16
MySQL database 5.0.67
PHP 5.2.13

I have a bunch of enabled modules, which I can list if needed, including Boost, and CKEditor.
Thank you for Mobile Tools, the Ipod theme is a winner, and let me know how I can help.

minoroffense’s picture

minoroffense commented
Status: Needs review » Postponed (maintainer needs more info)

Can you supply a patch/sample code/line numbers to investigate? Is this still an issue?

Thanks,

devin carlson’s picture

devin carlson commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

Please update to the latest version of Mobile Tools and Facebook Status and see if your problems persist.