Add page numbers to the administrative issue table of contents

Describe the problem you would like to solve
When Editors are ordering issue articles the natural order is the page order of the articles. However, the current table of contents ordering view does not show the page numbers.

Describe the solution you’d like
Add page numbers to the toc.

Who is asking for this feature?
Editors

Hi,

I agree that this would be very handy for a lot of editors and journals that use page numbers and voted for it.

I’ve implemented a patch in our OJS systems. Small changes need to be added to controllers/grid/toc/TocGridCellProvider.php and controllers/grid/toc/TocGridHandler.php

controllers/grid/toc/TocGridCellProvider.php , around line 42

public function getTemplateVarsFromRowColumn($row, $column)
    {
        $element = $row->getData();
        $columnId = $column->getId();
        assert(!empty($columnId));
        # CHIMIA CHANGE 2021/12/19/mb display page number
        switch ($columnId) {
            case 'title':
                return ['label' => $element->getLocalizedTitle()];
            case 'pages':
                return array('label' => $element->getPages());
            case 'access':
                return ['selected' => $element->getCurrentPublication()->getData('accessStatus') == Submission::ARTICLE_ACCESS_OPEN];
            default: assert(false);
        }
    }

controllers/grid/toc/TocGridHandler.php, around line 92

        // Article title
        $this->addColumn(
            new GridColumn(
                'title',
                'article.title',
                null,
                null,
                $tocGridCellProvider
            )
        );

        # CHIMIA CHANGE 2021/12/19/mb Display page number
        // Article pages
        $this->addColumn(
            new GridColumn(
                'pages',
                'editor.issues.pages',
                null,
                null,
                $tocGridCellProvider
            )
        );
        # END CHIMIA CHANGE

Thanks, I have pretty much the same thing done locally.

ping @asmecher is this something that could be moved forward?