J4 Issue ?
avatar PhilETaylor
PhilETaylor
19 May 2018

Steps to reproduce the issue

Attempt to insert a CONTACT using the CMS Content dropdown in the editor in admin when editing a CUSTOM MODULE
Select the contact from the list in the modal

Expected result

The modal closes

Actual result

JS issue

Uncaught TypeError: window.parent.jModalClose is not a function

screen recording 2018-05-19 at 10 19 pm

System information (as much as possible)

79fd949

similar to #20496 BUT DIFFERENT ENOUGH to raise different issue as different error message and probable cause

avatar PhilETaylor PhilETaylor - open - 19 May 2018
avatar joomla-cms-bot joomla-cms-bot - change - 19 May 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 May 2018
avatar PhilETaylor PhilETaylor - change - 19 May 2018
The description was changed
avatar PhilETaylor PhilETaylor - edited - 19 May 2018
avatar PhilETaylor PhilETaylor - change - 19 May 2018
The description was changed
avatar PhilETaylor PhilETaylor - edited - 19 May 2018
avatar brianteeman brianteeman - change - 19 May 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-05-19 22:18:51
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 19 May 2018

Cannot replicate

avatar brianteeman brianteeman - close - 19 May 2018
avatar brianteeman brianteeman - change - 19 May 2018
Status Closed New
Closed_Date 2018-05-19 22:18:51
Closed_By brianteeman
avatar brianteeman brianteeman - reopen - 19 May 2018
avatar brianteeman
brianteeman - comment - 19 May 2018

oops pressed the wrong button

avatar PhilETaylor PhilETaylor - comment - 19 May 2018
avatar brianteeman
brianteeman - comment - 19 May 2018

can not replicate

contct

avatar dgrammatiko
dgrammatiko - comment - 19 May 2018

There is PR for this (although WIP, as we haven't decided yet the actual API): #19830

avatar PhilETaylor PhilETaylor - comment - 19 May 2018
avatar PhilETaylor PhilETaylor - comment - 19 May 2018
avatar dgrammatiko
dgrammatiko - comment - 19 May 2018

but this should work until the modals are redone.

So here is the actual situation: we removed the moo tools code and effectively the method jModalClose. We need a replacement for it and although it is painful to see things break, it is far more wise to get the API correctly for the entire life of J4.

avatar PhilETaylor PhilETaylor - comment - 19 May 2018
avatar dgrammatiko
dgrammatiko - comment - 19 May 2018

Other modals are closing ok - so there should be an easy fix to close this modal... like the others.

Not true, switch to codemirror and observe that none of the modals actually close. Anyways alpha is not about quick fixes, as I said the API needs to be the best we can provide as it's gonna last for the entire life of J4

avatar PhilETaylor PhilETaylor - comment - 19 May 2018
avatar dgrammatiko
dgrammatiko - comment - 19 May 2018

#19830 is still open once this get's merged all modals will be fixed, The PR you mentioned was the one that ended up breaking because I was solving too many issues at one pr.

avatar mbabker
mbabker - comment - 19 May 2018

We can't keep applying bandaid quick fixes to keep something usable until its replacement is fully ready, we don't have the time or resources to be doing that. We know it's broken, we know what the replacement is, help test/review said replacement so the issue gets fixed, or look for the dozen or so of us working on all these things to have less time to spend on things because we're working on bandaid quick fixes.

avatar franz-wohlkoenig franz-wohlkoenig - change - 20 May 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 May 2018
Category com_contact JavaScript
avatar brianteeman brianteeman - change - 20 May 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 20 May 2018
avatar brianteeman
brianteeman - comment - 31 May 2018

Closing this as it will be resolved by #19830

avatar brianteeman brianteeman - change - 31 May 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-05-31 13:14:54
Closed_By brianteeman
avatar brianteeman brianteeman - close - 31 May 2018

Add a Comment

Login with GitHub to post a comment