Project

General

Profile

Actions

Feature #19152

closed

Remove JS popup when login expired

Added by Christoph Koehler over 16 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2008-07-25
Due date:
% Done:

0%

Estimated time:
PHP Version:
5.2
Tags:
Complexity:
Sprint Focus:

Description

As part of 4.3 I implemented this idea.
Patch is attached, please provide feedback.

This is how it works now:
30 seconds before the login expires, a modalbox window slides down with a countdown of 30 seconds and a button to refresh the login. This does not require username/password, it simply extends the current session.

Once the countdown expires, a login form is automatically loaded. Login will refresh the session and the internal timer and allow the user to continue work as before.
(issue imported from #M9052)


Files

modalbox.zip (15.7 KB) modalbox.zip Administrator Admin, 2008-07-25 17:59
error popup.png (248 KB) error popup.png Administrator Admin, 2008-07-25 21:30
9052_modal_login_7.diff (24.3 KB) 9052_modal_login_7.diff Administrator Admin, 2008-11-04 08:44
9052_modal_login_13.diff (26.3 KB) 9052_modal_login_13.diff Administrator Admin, 2009-01-08 18:11

Related issues 7 (0 open7 closed)

Related to TYPO3 Core - Feature #19586: After auto Logout the Clear Cache function stil appears to be working, but of course it does nothingClosedChristian Kuhn2008-11-10

Actions
Related to TYPO3 Core - Bug #19841: Backend not usable in Internet Explorer anymoreClosedOliver Hader2009-01-16

Actions
Related to TYPO3 Core - Bug #19897: Bad interaction between OpenId auth and ExtJS relogin boxClosedOliver Hader2009-01-23

Actions
Related to TYPO3 Core - Feature #15422: Make "Login-error or session timed-out" - page configurableClosedFrancois Suter2006-01-15

Actions
Related to TYPO3 Core - Feature #15631: BE should reload when a new session with another user is createdClosedChris topher2006-02-15

Actions
Related to TYPO3 Core - Bug #18944: "Clear Cache" commands in toolbar not complaining when session has expiredClosedChris topher2008-06-12

Actions
Related to TYPO3 Core - Bug #33759: AJAX login doesn't work with RSAauth and Salted passClosed2012-02-08

Actions
Actions #1

Updated by Christoph Koehler over 16 years ago

Changed patch file to exclude new files and attached the new files as a .zip. They need to go into typo3/contrib.

Actions #2

Updated by Ingo Renner over 16 years ago

what license is modal box?

Actions #3

Updated by Christoph Koehler over 16 years ago

MIT. There is a bug with IE (imagine that) that I am still trying to figure out.

Actions #4

Updated by Ingo Renner over 16 years ago

MIT... so most likely it's actually X11, but this should be ok with the GPL anyways

Actions #5

Updated by Ingo Renner over 16 years ago

some points about your patch:

  • the class should go into typo3/classes/
  • the license header is missing
  • phpDoc is missing
  • xclass definition is missing
  • don't use tables for layout
  • don't use inline html (though it is debatable here)
  • the patch doesn't apply directly as it's not relative to the TYPO3 root directory (a, b at the beginning)
  • include the modal box sources in the patch
  • try using TYPO3's spinner, there should be one in typo3/gfx/ or so
Actions #6

Updated by Ingo Renner over 16 years ago

... ok the spinner can stay, looks good.

However, I get an error as in the attached scrrenshot

Actions #7

Updated by Christoph Koehler over 16 years ago

Thanks, will look at those.

tables for layout: Copied and pasted the html from the current login box in index.php, but will change that.

inline html: I assume you mean the return values of the ajax methods. We can talk about that.

patch: yeah, will change that, too.

modalbox source: benni asked me to zip it since it contains binary files (the spinner)

spinner: it's included in the modalbox, i would have to change the source to switch.

Actions #8

Updated by Christoph Koehler over 16 years ago

hmm odd. Works fine in Safari here. Wonder if the patch is borked...

Actions #9

Updated by Benni Mack over 16 years ago

Hey Christoph,

my first remarks:

a) Your new class and all its methods does not have any comments nor License infos. Please add them.
b) Also, use PHP5 style methods (public / protected) for new functions
c) Don't echo anything directly. The AJAXObj will handle the output. Have a look at the TYPO3AJAX class and how it is used in other places in the core on how to return code as you want it.
d) Don't know if you need to urlencode this: "ajaxID=BackendLogin%3A%3AForm'

Actions #10

Updated by Ingo Renner over 16 years ago

more comments:

  • avoid inline css styles, put them into classes
  • try to stay within 80 characters/line
Actions #11

Updated by Christoph Koehler over 16 years ago

Added file 9052_modal_login_2.diff with the newest changes.

I took most of the suggestions here into account.

I also changed the way that TBE_EDITOR determines a timeout. This will need some testing.
I changed jsfunc.tbe_editor.js slightly to use the busy.busyloadTime instead of the TBE EDITOR internal time, making it a little more unified.
It seems to work properly so that now, when you are editing a record and your login is timing out, once you relogin, the warning that your login expired will not show anymore.

Apply the patch and stick the files from the modalbox.zip archive into the contrib directory, so that it will be available from typo3/contrib/modalbox/modalbox.js.

Actions #12

Updated by Christoph Koehler over 16 years ago

New patch file with corrected paths. Sorry about that.

Actions #13

Updated by Christoph Koehler over 16 years ago

Hopefully the last version. Fixed some IE 6 and 7 issues and found it working well.

Actions #14

Updated by Christoph Koehler about 16 years ago

New patch, now using ExtJS. Basic functionality is the same.

Actions #15

Updated by Jeff Segars about 16 years ago

Added v6 of the patch that applies cleanly to trunk.

Actions #16

Updated by Steffen Kamper about 16 years ago

I added logout buttons and Christoph added a missing line, see patch v10

Actions #17

Updated by Stefan Geith almost 16 years ago

Last patch (v10) doesn't work - seems it doens'thave the correct patch-format.
Could you please re-add a working patch ? Thanks !

Actions #18

Updated by Oliver Hader almost 16 years ago

Committed to SVN Trunk (rev. 4645) by Dmitry.

Actions

Also available in: Atom PDF