Now that the node frontpage listing and taxonomy pages are views the node.settings::items_per_page configuration is no longer used and should be removed. This value is set using a form alter. Form alters that save config are being converted in #2392319: Config objects (but not config entities) should by default be immutable to objects. This removal should happen first.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue priority Critical because it blocks #2392319: Config objects (but not config entities) should by default be immutable
Disruption Not disruptive since it is not used in core. If contrib is using the old variable / new config it will need to be updated.
CommentFileSizeAuthor
#1 2401035.1.patch4.48 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Issue summary: View changes
Status: Active » Needs review
FileSize
2401035.1.patch4.48 KB

Patch to remove this.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran commented
Status: Needs review » Reviewed & tested by the community

Fair enough.

swentel’s picture

swentel CreditAttribution: swentel commented
Status: Reviewed & tested by the community » Needs review

There's still code in migrate that refers to this, should this be removed ?

core/modules/migrate_drupal/src/Tests/Dump/Drupal6NodeSettings.php:      'name' => 'default_nodes_main',
core/modules/migrate_drupal/config/install/migrate.migration.d6_node_settings.yml:    - default_nodes_main
core/modules/migrate_drupal/config/install/migrate.migration.d6_node_settings.yml:  items_per_page: default_nodes_main
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented

@swentel yep - doesn't the patch in #1 do that?

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Applied the patch and went through the leftover instance of 'items_per_page' and they all have been related to either block settings or views, so we are fine here.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 451bf65 on 8.0.x 
    Issue #2401035 by alexpott: items_per_page in node.settings is no longer...
swentel’s picture

swentel CreditAttribution: swentel commented

Ugh right, sorry, grepped after I had reset my environment - was closing a lot duplicate issues regarding this setting in the queue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.