From 96a8e4b6dc9d55f18636156e15c47f89b69d271b Mon Sep 17 00:00:00 2001 From: Carlos Ruiz Date: Tue, 28 May 2013 12:43:54 -0500 Subject: [PATCH] IDEMPIERE-970 Role with the can Export option deselected would actually be able to export / Peer review --- .../org/adempiere/webui/window/ZkReportViewer.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/window/ZkReportViewer.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/window/ZkReportViewer.java index 53f0e729d5..f29419248d 100644 --- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/window/ZkReportViewer.java +++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/window/ZkReportViewer.java @@ -252,16 +252,16 @@ public class ZkReportViewer extends Window implements EventListener, ITab ? MSysConfig.getValue(MSysConfig.ZK_REPORT_FORM_OUTPUT_TYPE,"PDF",Env.getAD_Client_ID(m_ctx),Env.getAD_Org_ID(m_ctx)) : MSysConfig.getValue(MSysConfig.ZK_REPORT_TABLE_OUTPUT_TYPE,"PDF",Env.getAD_Client_ID(m_ctx),Env.getAD_Org_ID(m_ctx)); - if ("HTML".equals(type)) + if ("HTML".equals(type)) { previewType.setSelectedIndex(0); - else if ("PDF".equals(type)) + } else if ("PDF".equals(type) && m_isCanExport) { previewType.setSelectedIndex(1); - else if ("XLS".equals(type)) + } else if ("XLS".equals(type) && m_isCanExport) { previewType.setSelectedIndex(2); - else + } else { // XXX - provide hint if unexpected value - previewType.setSelectedIndex(0); //fall back to PDF - + previewType.setSelectedIndex(0); //fall back to HTML + } labelDrill.setValue(Msg.getMsg(Env.getCtx(), "Drill") + ": "); toolBar.appendChild(labelDrill);