Skip to content
This repository has been archived by the owner on Mar 8, 2021. It is now read-only.

Quick Manager+ not holding a lock #1120

Closed
bossloper opened this issue Dec 16, 2016 · 7 comments
Closed

Quick Manager+ not holding a lock #1120

bossloper opened this issue Dec 16, 2016 · 7 comments

Comments

@bossloper
Copy link
Contributor

If manager A edits a resource using Quick Manager+ and holds the edit open longer than the 'session timeout' length then the resource becomes unlocked and can be edited by manager B at the same time.

Presuming the 'mailer ping' is not running when using Quick Manager+

@Deesen
Copy link
Contributor

Deesen commented Dec 17, 2016

Hmm but the mailer ping does work as long as you have the main manager window open, or do you close this window meanwhile? As long as this session is valid, locks should not get removed by manager B. I need to check in detail as per your explanation.

@bossloper
Copy link
Contributor Author

Quick Manager+ does not run the top frame though. (I believe?!) So if you are just on the frontend editing a resource it will only be running the main manager window. It is the mainmenu frame that is doing the ping?

@Deesen
Copy link
Contributor

Deesen commented Dec 17, 2016

I mean, me I always work with tabs, and I always have a tab with backend and most times one with frontend open. Means I always have a ping, also when using QM.

@bossloper
Copy link
Contributor Author

Yes, for us 'heavy users'. But I have sites where some managers are only on the frontend, some even have 'Manager Login Startup' page set on the manager 'user' tab so after login they never see the backend.

@Deesen
Copy link
Contributor

Deesen commented Dec 17, 2016

Interesting, thanks for the insight. Then QM needs its own ping-script.

Deesen added a commit to Deesen/evolution that referenced this issue Dec 19, 2016
@Deesen
Copy link
Contributor

Deesen commented Dec 20, 2016

Should work now too.

@SerNeo
Copy link

SerNeo commented Jan 18, 2017

Hi,
I had an error in console: Uncaught TypeError: $ is not a function it was in line:

// Assure keeping session and related locks alive
$(document).ready(function($) { ...

To fix this error I had changed to:

jQuery(function ($) { ..

Hope this will be fixed in new release.

Dmi3yy added a commit that referenced this issue Nov 25, 2019
2.0 Поправил лого для темы .lightness
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants