Move the pageSize setting to the JSP level

Setting the pageSize as a global variable in the window object from Jade
doesn't work. Moving it to the layout_head.jsp with the other variables
ensures it is defined when the ReactJS code is used.

java/code/src/com/suse/manager/webui/controllers/contentmanagement/templates/list-filters.jade

@@ -2,7 +2,6 @@ include /templates/common.jade
 
 +userRoles
 +userLocalization
-+userPreferences
 
 #content-management-list-filters
 

java/code/src/com/suse/manager/webui/controllers/contentmanagement/templates/list-projects.jade

@@ -1,7 +1,6 @@
 include /templates/common.jade
 
 +userRoles
-+userPreferences
 
 #content-management-list-projects
 

java/code/src/com/suse/manager/webui/templates/audit/cve.jade

@@ -5,8 +5,6 @@ include ../common.jade
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('audit/cveaudit')
         .then(function(module) { module.renderer('cveaudit') });

java/code/src/com/suse/manager/webui/templates/common.jade

@@ -5,10 +5,6 @@ mixin csrfToken
   input(type="hidden", name="csrf_token", value="#{csrf_token}")
 
 
-mixin userPreferences
-  script(type='text/javascript').
-      window.userPrefPageSize = #{pageSize};
-
 mixin userRoles
     script(type='text/javascript').
         window.global_userRoles = !{roles};

java/code/src/com/suse/manager/webui/templates/content_management/list-profiles.jade

@@ -6,8 +6,6 @@ script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
     window.isAdmin = #{isAdmin};
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('images/image-profiles')
         .then(function(module) { module.renderer('image-profiles') });

java/code/src/com/suse/manager/webui/templates/content_management/list-stores.jade

@@ -6,8 +6,6 @@ script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
     window.isAdmin = #{is_admin};
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('images/image-stores')
         .then(function(module) { module.renderer('image-stores') });

java/code/src/com/suse/manager/webui/templates/content_management/view.jade

@@ -11,8 +11,6 @@ script(type='text/javascript').
     window.isRuntimeInfoEnabled = #{isRuntimeInfoEnabled};
     window.osImageStoreUrl = "#{osImageStoreUrl}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('images/image-view')
         .then(function(module) { module.renderer('image-view') });

java/code/src/com/suse/manager/webui/templates/formula_catalog/list.jade

@@ -2,8 +2,6 @@ include ../common.jade
 
 #formula-catalog
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('salt/formula-catalog/org-formula-catalog')
         .then(function(module) {

java/code/src/com/suse/manager/webui/templates/minion/list.jade

@@ -5,7 +5,6 @@ include ../common.jade
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
     window.serverId = "#{serverId}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('salt/keys/key-management')

java/code/src/com/suse/manager/webui/templates/notification-messages/list.jade

@@ -7,8 +7,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('notifications/notification-messages')
         .then(function(module) { module.renderer('notification-messages') });

java/code/src/com/suse/manager/webui/templates/packages/list.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('packages/list')

java/code/src/com/suse/manager/webui/templates/products/show.jade

@@ -9,8 +9,6 @@ script(type='text/javascript').
     window.scc_refresh_file_locked_status = #{refreshFileLocked};
     window.noToolsChannelSubscription_flag_from_backend = #{noToolsChannelSubscription};
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('admin/setup/products/products')
         .then(function(module) { module.renderer('products') });

java/code/src/com/suse/manager/webui/templates/schedule/maintenance-windows.jade

@@ -9,8 +9,6 @@ script(type='text/javascript').
     window.type = "#{type}";
     window.isAdmin = "#{isAdmin}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('maintenance/maintenance-windows')
         .then(function (module) { module.renderer('maintenance-windows') });

java/code/src/com/suse/manager/webui/templates/schedule/recurring-actions.jade

@@ -9,8 +9,6 @@ script(type='text/javascript').
     window.timezone = "#{h.renderTimezone()}";
     window.localTime = "#{h.renderLocalTime()}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('recurring/recurring-actions')
         .then(function (module) { module.renderer('recurring-actions') });

java/code/src/com/suse/manager/webui/templates/subscription-matching/show.jade

@@ -4,7 +4,6 @@ include ../common.jade
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('audit/subscription-matching')

java/code/src/com/suse/manager/webui/templates/systems/all-list.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('systems/list/all')

java/code/src/com/suse/manager/webui/templates/systems/virtual-list.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('systems/list/virtual')

java/code/src/com/suse/manager/webui/templates/taskotop/show.jade

@@ -4,8 +4,6 @@ include ../common.jade
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('admin/task-engine-status/taskotop')
         .then(function(module) { module.renderer('taskotop') });

java/code/src/com/suse/manager/webui/templates/virtualhostmanager/list.jade

@@ -4,8 +4,6 @@ include ../common.jade
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
 
-+userPreferences
-
 script(type='text/javascript').
     spaImportReactPage('systems/virtualhostmanager/virtualhostmanager')
         .then(function(module) { module.renderer('virtual-host-managers') });

java/code/src/com/suse/manager/webui/templates/virtualization/guests/create.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/guests/create/guests-create')

java/code/src/com/suse/manager/webui/templates/virtualization/guests/edit.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/guests/edit/guests-edit')

java/code/src/com/suse/manager/webui/templates/virtualization/guests/show.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/guests/list/guests-list')
@@ -14,7 +13,6 @@ script(type='text/javascript').
               'virtualguests',
               {
                   serverId: "#{server.id}",
-                  pageSize: "#{pageSize}",
                   saltEntitled: #{salt_entitled},
                   foreignEntitled: #{foreign_entitled},
                   isAdmin: #{is_admin},

java/code/src/com/suse/manager/webui/templates/virtualization/nets/create.jade

@@ -6,7 +6,6 @@ script(src='/javascript/validator/validator.min.js?cb=#{webBuildtimestamp}', typ
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/nets/create/nets-create')

java/code/src/com/suse/manager/webui/templates/virtualization/nets/edit.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/nets/edit/nets-edit')

java/code/src/com/suse/manager/webui/templates/virtualization/nets/show.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/nets/list/nets-list')
@@ -14,7 +13,6 @@ script(type='text/javascript').
               'virtualnets',
               {
                   serverId: "#{server.id}",
-                  pageSize: "#{pageSize}",
                   hypervisor: "#{hypervisor}",
                   support_enhanced_network: #{support_enhanced_network}
               }

java/code/src/com/suse/manager/webui/templates/virtualization/pools/create.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/pools/create/pools-create')

java/code/src/com/suse/manager/webui/templates/virtualization/pools/edit.jade

@@ -5,7 +5,6 @@ script(src='/javascript/momentjs/moment-with-langs.min.js?cb=#{webBuildtimestamp
 
 script(type='text/javascript').
     window.csrfToken = "#{csrf_token}";
-+userPreferences
 
 script(type='text/javascript').
     spaImportReactPage('virtualization/pools/edit/pools-edit')

java/code/src/com/suse/manager/webui/templates/virtualization/pools/show.jade

@@ -13,7 +13,6 @@ script(type='text/javascript').
               'virtualpools',
               {
                   serverId: "#{server.id}",
-                  pageSize: "#{pageSize}",
                   hypervisor: "#{hypervisor}"
               }
             )

java/code/src/com/suse/manager/webui/utils/UserPreferenceUtils.java

@@ -204,4 +204,16 @@ public String getUserDateFormat(PageContext pageContext) {
     public String getUserTimeFormat(PageContext pageContext) {
         return "HH:mm";
     }
+
+    /**
+     * @param pageContext the current PageContext
+     * @return the user preferred page size
+     */
+    public int getUserPageSize(PageContext pageContext) {
+        User user = getAuthenticatedUser(pageContext);
+        if (isUserAuthenticated(user)) {
+            return user.getPageSize();
+        }
+        return 15;
+    }
 }

java/code/webapp/WEB-INF/decorators/layout_head.jsp

@@ -59,12 +59,14 @@
     <c:set var="userTimeZone" value="${GlobalInstanceHolder.USER_PREFERENCE_UTILS.getExtendedUserTimeZone(pageContext)}"/>
     <c:set var="userDateFormat" value="${GlobalInstanceHolder.USER_PREFERENCE_UTILS.getUserDateFormat(pageContext)}"/>
     <c:set var="userTimeFormat" value="${GlobalInstanceHolder.USER_PREFERENCE_UTILS.getUserTimeFormat(pageContext)}"/>
+    <c:set var="userPageSize" value="${GlobalInstanceHolder.USER_PREFERENCE_UTILS.getUserPageSize(pageContext)}"/>
     <script>
         window.serverTime='${serverTime}'
         window.serverTimeZone='${serverTimeZone}'
         window.userTimeZone='${userTimeZone}'
         window.userDateFormat='${userDateFormat}'
         window.userTimeFormat='${userTimeFormat}'
+        window.userPrefPageSize=${userPageSize}
     </script>
 
     <script src="/javascript/legacy/jquery.min.js?cb=${cb_version}"></script>

java/spacewalk-java.changes.cbosdo.page-size

@@ -0,0 +1 @@
+- Fix the use of page size preference in systems and packages lists

web/html/src/manager/virtualization/ListTab.tsx

@@ -31,7 +31,6 @@ type CreateModalButtonType = (actionType: string, actionData: Array<ModalDataTyp
 type Props = {
   serverId: string;
   saltEntitled: boolean;
-  pageSize: number;
   type: string;
   urlType?: string;
   title: string;
@@ -230,7 +229,6 @@ export function ListTab(props: Props) {
                     emptyText={t(`No virtual ${props.type} to show.`)}
                     identifier={(row) => row[props.idName]}
                     initialSortColumnKey="name"
-                    initialItemsPerPage={props.pageSize}
                     selectable
                     selectedItems={selectedItems.map((item) => item[props.idName])}
                     onSelect={(items) => setSelectedItems(data?.filter((d) => items.includes(d[props.idName])) || [])}

web/html/src/manager/virtualization/guests/list/guests-list.renderer.tsx

@@ -7,21 +7,16 @@ import { HostInfo } from "./guests-list";
 
 type RendererProps = {
   serverId: string;
-  pageSize: number;
   saltEntitled: boolean;
   foreignEntitled: boolean;
   isAdmin: boolean;
   hostInfo?: HostInfo;
 };
 
-export const renderer = (
-  id: string,
-  { serverId, pageSize, saltEntitled, foreignEntitled, isAdmin, hostInfo }: RendererProps
-) => {
+export const renderer = (id: string, { serverId, saltEntitled, foreignEntitled, isAdmin, hostInfo }: RendererProps) => {
   SpaRenderer.renderNavigationReact(
     <GuestsList
       serverId={serverId}
-      pageSize={pageSize}
       saltEntitled={saltEntitled}
       foreignEntitled={foreignEntitled}
       isAdmin={isAdmin}

web/html/src/manager/virtualization/guests/list/guests-list.tsx

@@ -19,7 +19,6 @@ export type HostInfo = {
 
 type Props = {
   serverId: string;
-  pageSize: number;
   saltEntitled: boolean;
   foreignEntitled: boolean;
   isAdmin: boolean;
@@ -99,7 +98,6 @@ export function GuestsList(props: Props) {
       <ListTab
         serverId={props.serverId}
         saltEntitled={props.saltEntitled}
-        pageSize={props.pageSize}
         type="guest"
         title={t("Hosted Virtual Systems")}
         description={t("This is a list of virtual guests which are configured to run on this host.")}

web/html/src/manager/virtualization/nets/list/nets-list.renderer.tsx

@@ -6,19 +6,13 @@ import { NetsList } from "./nets-list";
 
 type RendererProps = {
   serverId: string;
-  pageSize: number;
   hypervisor: string;
   support_enhanced_network: boolean;
 };
 
-export const renderer = (id: string, { serverId, pageSize, hypervisor, support_enhanced_network }: RendererProps) => {
+export const renderer = (id: string, { serverId, hypervisor, support_enhanced_network }: RendererProps) => {
   SpaRenderer.renderNavigationReact(
-    <NetsList
-      serverId={serverId}
-      pageSize={pageSize}
-      hypervisor={hypervisor}
-      allow_changing={support_enhanced_network}
-    />,
+    <NetsList serverId={serverId} hypervisor={hypervisor} allow_changing={support_enhanced_network} />,
     document.getElementById(id)
   );
 };

web/html/src/manager/virtualization/nets/list/nets-list.tsx

@@ -12,7 +12,6 @@ import { ListTab } from "../../ListTab";
 
 type Props = {
   serverId: string;
-  pageSize: number;
   hypervisor: string;
   allow_changing: boolean;
 };
@@ -46,7 +45,6 @@ export function NetsList(props: Props) {
       <HypervisorCheck hypervisor={props.hypervisor} />
       <ListTab
         serverId={props.serverId}
-        pageSize={props.pageSize}
         type="network"
         urlType="nets"
         title={t("Virtual Networks")}

web/html/src/manager/virtualization/pools/list/pools-list.renderer.tsx

@@ -6,13 +6,12 @@ import { PoolsList } from "./pools-list";
 
 type RendererProps = {
   serverId: string;
-  pageSize: number;
   hypervisor: string;
 };
 
-export const renderer = (id: string, { serverId, pageSize, hypervisor }: RendererProps) => {
+export const renderer = (id: string, { serverId, hypervisor }: RendererProps) => {
   SpaRenderer.renderNavigationReact(
-    <PoolsList pageSize={pageSize} serverId={serverId} hypervisor={hypervisor} />,
+    <PoolsList serverId={serverId} hypervisor={hypervisor} />,
     document.getElementById(id)
   );
 };