I have no idea what this could be, when i look in the firebug console i do not see a awnser from /user/login/ajax i also don't see html beeing send. Firebug does not show me any javascript errors. But when i click the login link in the demo the modal opens and i get a alert with:
An error occurred at /user/login/ajax.
Error Description:

CommentFileSizeAuthor
#11 863632-update-links.patch2.13 KBsolotandem
#9 dialog_863632.patch2.14 KBdrewish
#6 863632-fix-example-for-auth-users-6.patch2.19 KBamitaibu

Comments

Michsk’s picture

Michsk commented

this probably has to do with the jquery UI 1.7 issue.

Michsk’s picture

Michsk commented

had to do with ui 1.6

scotjam’s picture

scotjam commented

how do you get around this?

Michsk’s picture

Michsk commented

... your kidding right? Anyway read this issue good and you'll find out. I stoped using dialog api for it does not seem maintaned i switched to ctools moda.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

@lasac,
Dialog seems to be working ok for me.

@scotjam,
Make sure you try to login/ register an an *anon* user. Probably worth changing dialog_example not to allow those links for authenticated users...

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
Status: Active » Needs review
StatusFileSize
new 863632-fix-example-for-auth-users-6.patch2.19 KB

And here's the patch for dialog_example, that also removes white space.

Michsk’s picture

Michsk commented

the problem is that dialog_api does not work with jquery ui 1.7

Anonymous’s picture

Anonymous (not verified) commented

As I posted in #804002: Incompatible with jquery_ui 1.7?:

Some of this may have to do with jquery_update.

  1. When I use jquery_update 6.x-2.x-dev, dialog works.
  2. When I download with Drush, I of course get the supported version, 6.x-1.1. When using this version, none of the dialogs worked. When I then replaced 6.x-1.1 with the dev version, dialogs worked again.

As far as I could see, this isn't documented in any of the READMEs, but I may have missed it.

drewish’s picture

drewish commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)
StatusFileSize
new dialog_863632.patch2.14 KB

committed the attached to DRUPAL-6--1. bumping to the 7.x branch so someone can see if it applies there.

moonray’s picture

moonray commented
Status: Patch (to be ported) » Needs work

The patch won't work as is. Some of the functions to use have changed.

solotandem’s picture

solotandem commented
Status: Needs work » Needs review
StatusFileSize
new 863632-update-links.patch2.13 KB

Updated patch for D7.14 and latest code for Dialog.

See also patch in #5 of #1457230: Login or Register does not validate. 7.x-1.x-dev that fixes the form submission for login and register. The request new password form still does not display.

mustanggb’s picture

mustanggb commented

I did the same thing but used user_is_anonymous() instead.

Also added a user_access('access content') check to "Simple test".

devin carlson’s picture

devin carlson commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

The 7.x-1.x branch of the Dialog module is unsupported.