Skip to content
Snippets Groups Projects
Commit eaa4fe66 authored by srosse's avatar srosse
Browse files

OO-986: fix resume on dynamic tab

parent b074a8b5
No related branches found
No related tags found
No related merge requests found
...@@ -294,8 +294,7 @@ public class BaseFullWebappController extends BasicController implements ChiefCo ...@@ -294,8 +294,7 @@ public class BaseFullWebappController extends BasicController implements ChiefCo
// Inline translation interceptor. when the translation tool is enabled it // Inline translation interceptor. when the translation tool is enabled it
// will start the translation tool in translation mode, if the overlay // will start the translation tool in translation mode, if the overlay
// feature is enabled it will start in customizing mode // feature is enabled it will start in customizing mode
// fxdiff: allow user-managers to use the inline translation also. TODO: // fxdiff: allow user-managers to use the inline translation also.
// do this with a proper right-mgmt!
if (ureq.getUserSession().isAuthenticated() if (ureq.getUserSession().isAuthenticated()
&& (ureq.getUserSession().getRoles().isOLATAdmin() || ureq.getUserSession().getRoles().isUserManager()) && (ureq.getUserSession().getRoles().isOLATAdmin() || ureq.getUserSession().getRoles().isUserManager())
&& (I18nModule.isTransToolEnabled() || I18nModule.isOverlayEnabled())) { && (I18nModule.isTransToolEnabled() || I18nModule.isOverlayEnabled())) {
...@@ -461,7 +460,7 @@ public class BaseFullWebappController extends BasicController implements ChiefCo ...@@ -461,7 +460,7 @@ public class BaseFullWebappController extends BasicController implements ChiefCo
private void initializeDefaultSite(UserRequest ureq) { private void initializeDefaultSite(UserRequest ureq) {
if (sites != null && sites.size() > 0 if (sites != null && sites.size() > 0
&& curSite == null && curSite == null && curDTab == null
&& contentCtrl == null) { && contentCtrl == null) {
SiteInstance s = sites.get(0); SiteInstance s = sites.get(0);
//activate site only if no content was set -> allow content before activation of default site. //activate site only if no content was set -> allow content before activation of default site.
...@@ -1168,7 +1167,7 @@ public class BaseFullWebappController extends BasicController implements ChiefCo ...@@ -1168,7 +1167,7 @@ public class BaseFullWebappController extends BasicController implements ChiefCo
* *
* @return * @return
*/ */
public boolean isCanCloseDTab(DTab dtab) { public boolean isCanCloseDTab() {
//can close //can close
return (sites != null && !sites.isEmpty()) || (dtabs != null && dtabs.size() > 1); return (sites != null && !sites.isEmpty()) || (dtabs != null && dtabs.size() > 1);
} }
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<li class="#if($tabhelper.isDTabActive($dtab)) active #end $!navel.getIconCSSClass()"> <li class="#if($tabhelper.isDTabActive($dtab)) active #end $!navel.getIconCSSClass()">
$r.render("a$postfix") $r.render("a$postfix")
#if($tabhelper.isCanCloseDTab($dtab)) #if($tabhelper.isCanCloseDTab())
$r.render("c$postfix") ## close link has class b_nav_tab_close $r.render("c$postfix") ## close link has class b_nav_tab_close
#end #end
</li> </li>
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment