Contextual link is not shown when only "Use contextual links" and "Access draggable views" permissions given.

There is a permission "Access draggable views" and it should restrict from having possibility to sort. Please check it and advise if it doesn't work.

@supermoos I hope you don't test this with user no 1 as he passes all access checks.

Please try to logout (be anonymous user) and open same views page. Please advise whether you still can see ordering elements?

Also you can test login as another user and check whether he sees/doesn't see ordering elements depending on whether you give permission "Access draggable views" to authenticated users role.

I am sorry I don't understand what do you mean by "clicking cogwheel on block". I would suggest to test on clean installation and write explanation how to reproduce the problem.

Please come online to IRC #drupal-contribute channel and contact me to discuss your problem.

updated summary

Fixed. See commit http://drupalcode.org/project/draggableviews.git/commit/5f9bb2b

@supermoos You can clone from git repo (http://drupal.org/project/draggableviews/git-instructions). I would recommend reinstall module because I have added field api handler functionality.

Or you can wait till dev snapshot will be updated automatically.

I believe that in draggableviews_contextual_links_view_alter() the call to views_ui_contextual_links_suppress() is not necessary.

  // Do not add contextual link on view preview.
  if (!module_exists('views_ui')) {// && views_ui_contextual_links_suppress()) {
    return;
  }

If I comment out the call then the administer views permission is no longer required to render the context link for "Order View".

I do not think we need that piece of code at all that suppress the link and checks the view_ui module is enabled. Contextual links do not appear if view_ui are not enabled.

There is special type of Contextual links in the preview view of views. I think this got rewritten somewhere in the last couple of years.

Patch to remove the lines of code attached.

Should add that all the code after the check does not break or cause white screen of death that #1426810: DraggableViews gives the white screen of death when Views UI is disabled.

I see no reason for that code since the contextual link is not actually a views admin one it is a content admin one.

Actually I got a bit confused about what I was looking at in the previous comment.

The issue I have is different, the contextual links not displaying if views UI is disabled.

I think that is wrong because it is a content management link not a views UI link.

The patch in #16 works perfectly. All that's needed is to grant the "Access draggable views" permission.

#16 Committed.