Re: [Spacewalk-devel] All relevant errata as All Errata
On Friday 02 of March 2012 18:13:29 Duncan Mac-Vicar P. wrote: On 03/02/2012 05:55 PM, Duncan Mac-Vicar P. wrote: On 03/02/2012 04:06 PM, Tomas Lestach wrote: There's one more thing confusing for me, when we're in the errata area. I mean following two pages https://spacewalk/rhn/errata/Overview.do https://spacewalk/rhn/errata/RelevantErrata.do display the same errata content. We might want to drop one of them. Ok, I have a patch that gets rid of errata/Overview.do. I need to rebase for master. attached. Thank you! Committed as: 86be01047a59b3f0120e30ce1af8fbaebc9b75cf Is the removal of strings with this file as context needed? That would be really nice, even if we usually do not remove the unused strings. :( Duncan Regards, -- Tomas Lestach RHN Satellite Engineering, Red Hat ___ Spacewalk-devel mailing list Spacewalk-devel@redhat.com https://www.redhat.com/mailman/listinfo/spacewalk-devel
Re: [Spacewalk-devel] All relevant errata as All Errata
On 03/02/2012 04:06 PM, Tomas Lestach wrote: On Friday 02 of March 2012 15:41:49 Duncan Mac-Vicar P. wrote: Hi Astronauts, The errata hierarchy in the Errata tab is: Errata Relevant All Errata - Bugfix Errata - Enhancement Errata - Security Errata All All Errata - Bugfix Errata - Enhancement Errata - Security Errata When the user is in Relevant (Errata Relevant to Your System) having a subtab All Errata is confusing, because it really means Relevant Errata of all types, and not All Errata like the parent menu. I suggest to change it to All Types un both Errata-All and Errata-Relevant. Comments? I agree. Attached patch for english and spanish. In spanish I did not need to mention all TYPES as I can use the gender difference to make it very obvious if I am talking of erratas types or erratas. German, is still being discussed here in the office... Duncan From f6fcded8d4c718c1089d1aa31ee75196fd7a59db Mon Sep 17 00:00:00 2001 From: Duncan Mac-Vicar P dmacvi...@suse.de Date: Mon, 5 Mar 2012 13:27:53 +0100 Subject: [PATCH 1/2] Rename All Errata to All Types as we are refering to Erratas of all types (Bugfixes, Security) and it can be confused with the All Errata menu which refers to Relevant/All. See: https://www.redhat.com/archives/spacewalk-devel/2012-March/thread.html#2 --- .../frontend/strings/jsp/StringResource_en_US.xml |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_en_US.xml b/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_en_US.xml index 2cc91d6..becd5e4 100644 --- a/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_en_US.xml +++ b/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_en_US.xml @@ -3500,7 +3500,7 @@ button below, and lt;bgt;will be unable to log back inlt;/bgt;./source /context-group /trans-unit trans-unit id=erratalist.jsp.allerrata -sourceAll Errata/source +sourceAll Types/source context-group name=ctx context context-type=sourcefile/rhn/errata/AllErrata/context /context-group -- 1.7.7 From 27dae2a916f5aed5a59721492c38e625492db6c7 Mon Sep 17 00:00:00 2001 From: Duncan Mac-Vicar P dmacvi...@suse.de Date: Mon, 5 Mar 2012 13:32:49 +0100 Subject: [PATCH 2/2] Rename Toda la errata to Todos as we are refering to Erratas of all types (Bugfixes, Security) and it can be confused with the All Errata menu which refers to Relevant/All. See: https://www.redhat.com/archives/spacewalk-devel/2012-March/thread.html#2 (Spanish version) --- .../rhn/frontend/strings/jsp/StringResource_es.xml |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_es.xml b/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_es.xml index 813db7f..767f5b9 100644 --- a/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_es.xml +++ b/java/code/src/com/redhat/rhn/frontend/strings/jsp/StringResource_es.xml @@ -3491,7 +3491,7 @@ del rol de Administrador de la organización./target/trans-unit context-group name=ctx context context-type=sourcefile/rhn/errata/AllErrata/context /context-group - targetToda la errata/target/trans-unit + targetTodos/target/trans-unit !-- == AUDIT == -- trans-unit id=audit.overview.jsp.header -- 1.7.7 ___ Spacewalk-devel mailing list Spacewalk-devel@redhat.com https://www.redhat.com/mailman/listinfo/spacewalk-devel
Re: [Spacewalk-devel] All relevant errata as All Errata
On 03/02/2012 03:41 PM, Duncan Mac-Vicar P. wrote: I suggest to change it to All Types un both Errata-All and Errata-Relevant. +1 -- Miroslav Suchy Red Hat Satellite Engineering ___ Spacewalk-devel mailing list Spacewalk-devel@redhat.com https://www.redhat.com/mailman/listinfo/spacewalk-devel
Re: [Spacewalk-devel] All relevant errata as All Errata
On 03/02/2012 05:55 PM, Duncan Mac-Vicar P. wrote: On 03/02/2012 04:06 PM, Tomas Lestach wrote: There's one more thing confusing for me, when we're in the errata area. I mean following two pages https://spacewalk/rhn/errata/Overview.do https://spacewalk/rhn/errata/RelevantErrata.do display the same errata content. We might want to drop one of them. Ok, I have a patch that gets rid of errata/Overview.do. I need to rebase for master. attached. Is the removal of strings with this file as context needed? Duncan From 36ebcfc0f1be2cb7c193907f3a0f81716bb18771 Mon Sep 17 00:00:00 2001 From: Duncan Mac-Vicar P dmacvi...@suse.de Date: Fri, 2 Mar 2012 17:50:04 +0100 Subject: [PATCH] errata/Overview.do and errata/RelevantErrata.do provide the same list. Get rid of Overview and use RelevantErrata instead. See https://www.redhat.com/archives/spacewalk-devel/2012-March/msg3.html Conflicts: java/code/webapp/WEB-INF/nav/sitenav-authenticated.xml --- .../webapp/WEB-INF/nav/errata_overview_tabs.xml| 15 - .../webapp/WEB-INF/nav/sitenav-authenticated.xml |4 +- java/code/webapp/WEB-INF/pages/errata/overview.jsp | 27 --- java/code/webapp/WEB-INF/struts-config.xml | 34 +--- java/scripts/lwload.pl |2 +- 5 files changed, 4 insertions(+), 78 deletions(-) delete mode 100644 java/code/webapp/WEB-INF/nav/errata_overview_tabs.xml delete mode 100644 java/code/webapp/WEB-INF/pages/errata/overview.jsp diff --git a/java/code/webapp/WEB-INF/nav/errata_overview_tabs.xml b/java/code/webapp/WEB-INF/nav/errata_overview_tabs.xml deleted file mode 100644 index b259d08..000 --- a/java/code/webapp/WEB-INF/nav/errata_overview_tabs.xml +++ /dev/null @@ -1,15 +0,0 @@ -?xml version=1.0? -rhn-navi-tree label=errata_tabs invisible=1 formvar=prid title-depth=1 - rhn-tab name=erratalist.jsp.allerrata -rhn-tab-url/rhn/errata/Overview.do/rhn-tab-url - /rhn-tab - rhn-tab name=yourrhn.jsp.criticalsystems.bugfixerrata -rhn-tab-url/rhn/errata/OverviewBugErrata.do/rhn-tab-url - /rhn-tab - rhn-tab name=yourrhn.jsp.criticalsystems.enhancementerrata -rhn-tab-url/rhn/errata/OverviewEnhancementErrata.do/rhn-tab-url - /rhn-tab - rhn-tab name=yourrhn.jsp.criticalsystems.securityerrata -rhn-tab-url/rhn/errata/OverviewSecurityErrata.do/rhn-tab-url - /rhn-tab -/rhn-navi-tree diff --git a/java/code/webapp/WEB-INF/nav/sitenav-authenticated.xml b/java/code/webapp/WEB-INF/nav/sitenav-authenticated.xml index 49f9a2b..129cd55 100644 --- a/java/code/webapp/WEB-INF/nav/sitenav-authenticated.xml +++ b/java/code/webapp/WEB-INF/nav/sitenav-authenticated.xml @@ -109,8 +109,8 @@ /rhn-tab /rhn-tab /rhn-tab - rhn-tab name=Errata url=/rhn/errata/Overview.do active-image=tab-errata-selected.gif inactive-image=tab-errata.gif on-click=Sniglets::Lists-navi_empty_set node-id=target_systems_list -rhn-tab name=Errata url=/rhn/errata/Overview.do on-click=Sniglets::Lists-navi_empty_set node-id=target_systems_list + rhn-tab name=Errata url=/rhn/errata/RelevantErrata.do active-image=tab-errata-selected.gif inactive-image=tab-errata.gif on-click=Sniglets::Lists-navi_empty_set node-id=target_systems_list +rhn-tab name=Errata url=/rhn/errata/RelevantErrata.do on-click=Sniglets::Lists-navi_empty_set node-id=target_systems_list rhn-tab-directory/rhn/errata/rhn-tab-directory rhn-tab name=Relevant on-click=Sniglets::Lists-navi_empty_set node-id=target_systems_list rhn-tab-url/rhn/errata/RelevantErrata.do/rhn-tab-url diff --git a/java/code/webapp/WEB-INF/pages/errata/overview.jsp b/java/code/webapp/WEB-INF/pages/errata/overview.jsp deleted file mode 100644 index 6842a08..000 --- a/java/code/webapp/WEB-INF/pages/errata/overview.jsp +++ /dev/null @@ -1,27 +0,0 @@ -%@ taglib uri=http://rhn.redhat.com/rhn; prefix=rhn % -%@ taglib uri=http://rhn.redhat.com/tags/list; prefix=rl % -%@ taglib uri=http://java.sun.com/jsp/jstl/core; prefix=c % -%@ taglib uri=http://struts.apache.org/tags-html; prefix=html % -%@ taglib uri=http://struts.apache.org/tags-bean; prefix=bean % - -html:xhtml/ -html -head -meta name=page-decorator content=none / -/head -body -rhn:toolbar base=h1 img=/img/rhn-icon-errata.gif imgAlt=errata.overview.jsp.alt - helpUrl=/rhn/help/reference/en-US/s1-sm-errata.jsp - bean:message key=errata.overview.jsp.errataoverview/ -/rhn:toolbar - -rhn:dialogmenu mindepth=0 maxdepth=3 definition=/WEB-INF/nav/errata_overview_tabs.xml renderer=com.redhat.rhn.frontend.nav.DialognavRenderer / - -pbean:message key=errata.overview.jsp.summary//p - -h2bean:message key=errata.jsp.header//h2 - -%@ include file=/WEB-INF/pages/common/fragments/errata/relevant-errata-list.jspf % - -/body -/html diff --git a/java/code/webapp/WEB-INF/struts-config.xml b/java/code/webapp/WEB-INF/struts-config.xml index a6d7560..1d75f49 100644 --- a/java/code/webapp/WEB-INF/struts-config.xml +++ b/java/code/webapp/WEB-INF/struts-config.xml