[MPLUGIN-498] Move section rendering to separate methods · apache/maven-plugin-tools@fc41218 (original) (raw)

Original file line number Diff line number Diff line change
@@ -87,15 +87,28 @@ protected void renderBody() {
87 87 return;
88 88 }
89 89
90 -paragraph(getI18nString("goals.intro"));
91 -
92 90 boolean hasMavenReport = false;
93 91 for (MojoDescriptor mojo : pluginDescriptor.getMojos()) {
94 92 if (PluginUtils.isMavenReport(mojo.getImplementation(), project)) {
95 93 hasMavenReport = true;
94 +break;
96 95 }
97 96 }
98 97
98 +renderGoalsSection(hasMavenReport);
99 +
100 +renderSystemRequirementsSection();
101 +
102 +renderRequirementsHistoriesSection();
103 +
104 +renderUsageSection(hasMavenReport);
105 +
106 +endSection();
107 + }
108 +
109 +private void renderGoalsSection(boolean hasMavenReport) {
110 +paragraph(getI18nString("goals.intro"));
111 +
99 112 startTable();
100 113
101 114 String goalColumnName = getI18nString("goals.column.goal");
@@ -144,7 +157,9 @@ protected void renderBody() {
144 157 }
145 158
146 159 endTable();
160 + }
147 161
162 +private void renderSystemRequirementsSection() {
148 163 startSection(getI18nString("systemrequirements"));
149 164
150 165 paragraph(getI18nString("systemrequirements.intro"));
@@ -166,15 +181,9 @@ protected void renderBody() {
166 181 endTable();
167 182
168 183 endSection();
169 -
170 -renderRequirementsHistories();
171 -
172 -renderUsageSection(hasMavenReport);
173 -
174 -endSection();
175 184 }
176 185
177 -private void renderRequirementsHistories() {
186 +private void renderRequirementsHistoriesSection() {
178 187 if (requirementsHistories.isEmpty()) {
179 188 return;
180 189 }