[fixed]Lock timeout (fixed in ContentManager) not in DeMa...

Forum rules
Only administrators can post or move items here.
Post Reply
jce76350
Beta Tester
Beta Tester
Posts: 2033
Joined: Mon May 29, 2006 1:20 pm
Location: Rouen

[fixed]Lock timeout (fixed in ContentManager) not in DeMa...

Post by jce76350 »

(Rev SVN 9573)
In Settings - Global Settings - Advanced Setup
there is Lock timeout: xx
This option is used for locks to timeout in Content Manager

and in Settings - Design Manager
there is Lock Time out (minutes) : xx
this option is not used in Content Manager ?
Jean-Claude Etiemble
calguy1000
Support Guru
Support Guru
Posts: 8169
Joined: Tue Oct 19, 2004 6:44 pm
Location: Fernie British Columbia, Canada

Re: Lock timeout

Post by calguy1000 »

There are three settings:

a: in global settings
(this sets the 'default' lock timeout)

b: in content manager
(this sets the lock timeout for content pages, which overrides the default only for content pages).

c: in design manager
(this sets the lock timeout for templates, which overrides the default only for templates)
Follow me on twitter
Please post system information from "Extensions >> System Information" (there is a bbcode option) on all posts asking for assistance.
--------------------
If you can't bother explaining your problem well, you shouldn't expect much in the way of assistance.
jce76350
Beta Tester
Beta Tester
Posts: 2033
Joined: Mon May 29, 2006 1:20 pm
Location: Rouen

Re: Lock timeout

Post by jce76350 »

b: in content manager
(this sets the lock timeout for content pages, which overrides the default only for content pages).
Ok but
this option (in content manager) do not override the default in global settings
I test 10 in global settings and 5 in content manager
On content pages, lock a page and Expire = 10
Jean-Claude Etiemble
hoshy

Re: Lock timeout

Post by hoshy »

The problem is also described here

In the latest revision of the file /admin/ajax_lock.php (rev. 9443) the problem still exists:

Code: Select all

 // see if we can get this lock... if we can, it's just a touch
    $lock = null;
    try {
      $lock = CmsLock::load($type,$oid,$uid);
    }
    catch( CmsNoLockException $e ) {
      // lock doesn't exist, gotta create one.
      $lock = new CmsLock($type,$oid);
    }
 
As you can see the locking timeout is neither passed to CmsLock::load() nor to CmsLock() itself. In fact it does not even catch the values that might be passed from the modules via POST-request. See top of the file:

Code: Select all

$opt = get_parameter_value($_REQUEST,'opt','setup');
$type = get_parameter_value($_REQUEST,'type');
$oid = get_parameter_value($_REQUEST,'oid');
$uid = get_parameter_value($_REQUEST,'uid');
$lock_id = get_parameter_value($_REQUEST,'lock_id');
Anyway the lock timeout is not passed by the lockManager javascript. See file "/lib/jquery/js/jquery.cmsms_lock.js". I don't know where exactly the issue is but i can clearly see in the network console of firebug that there is no timeout passed in the ajax post request.
jce76350
Beta Tester
Beta Tester
Posts: 2033
Joined: Mon May 29, 2006 1:20 pm
Location: Rouen

Re: Lock timeout

Post by jce76350 »

OK Solved for the locking into CMSContentManager (rev 9577)

But not Ok for the locking into DesignManager
Global Settings - Advanced Setup Lock timeout: 8
Settings - Design Manager Lock timeout: 5
so
in Design Manager lock a template Expires: 60
in Design Manager lock a Stylesheet Expires: 5
Jean-Claude Etiemble
jce76350
Beta Tester
Beta Tester
Posts: 2033
Joined: Mon May 29, 2006 1:20 pm
Location: Rouen

Re: Lock timeout (fixed in ContentManager) not in Design Man

Post by jce76350 »

In Design Manager / Stylesheets if a stylesheet is locked there is no icon warning.gif on the line locked
Jean-Claude Etiemble
uniqu3

Re: Lock timeout (fixed in ContentManager) not in Design Man

Post by uniqu3 »

jce76350 wrote:In Design Manager / Stylesheets if a stylesheet is locked there is no icon warning.gif on the line locked
Fixed in SVN rev: 9611
jce76350
Beta Tester
Beta Tester
Posts: 2033
Joined: Mon May 29, 2006 1:20 pm
Location: Rouen

Re: Lock timeout (fixed in ContentManager) not in Design Man

Post by jce76350 »

In Design Manager / Stylesheets if a stylesheet is locked there is no icon warning.gif on the line locked
Fixed in SVN rev: 9611
@ uniqu3 : Whaooo super quick fix

Now for calguy1000
http://forum.cmsmadesimple.org/viewtopi ... 28#p312528
But not Ok for the locking into DesignManager
Global Settings - Advanced Setup Lock timeout: 8
Settings - Design Manager Lock timeout: 5
so
in Design Manager lock a template Expires: 60
Jean-Claude Etiemble
uniqu3

Re: Lock timeout (fixed in ContentManager) not in Design Man

Post by uniqu3 »

Fixed in SVN rev 9624

- Lock timeout on templates should work properly now.
- Edit actions are locked with exception error message on direct url access attempt
- If for whatever reason a edit action interface is accessed by direct url, submit and apply is disabled with disabled attribute (that should be enough for a regular user who doesn't want to do any harm)
- If a lock has expired but user has left the page open, that user should no longer be able to continue working, on apply or submit, exception error will be shown.

I didn't check CotnentManager though ;)
jce76350
Beta Tester
Beta Tester
Posts: 2033
Joined: Mon May 29, 2006 1:20 pm
Location: Rouen

Re: [fixed]Lock timeout (fixed in ContentManager) not in DeM

Post by jce76350 »

everything : content and desing seems ok for me :)
Jean-Claude Etiemble
Post Reply

Return to “Closed Issues”