Move _contextual_id_to_links() to a helper class to simplify testing.

Created on 30 April 2017, about 7 years ago
Updated 9 March 2023, over 1 year ago

As this issue completed.

#2869903: ContextualController: remove ContainerAwareTrait →

There was this insight

We should file a an new issue to move _contextual_id_to_links() into this controller. The controller is the only place this is used. And it makes the controller not unit testable.

There is also a maintenance hazard - in relation to php7.2 - http://php.net/manual/en/function.parse-str.php

for the php_parse() function we must always supply a second parameter

Warning
Using this function without the result parameter is highly DISCOURAGED and DEPRECATED as of PHP 7.2.

📌 Task
Status

Needs work

Version

10.1 ✨

Component
Contextual  →

Last updated 6 days ago

  • Maintained by
  • 🇦🇺Australia @larowlan
Created by

🇬🇧United Kingdom martin107

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024