Remove dependency on Prepopulate

Created on 19 March 2024, 8 months ago
Updated 4 June 2024, 6 months ago

The Prepopulate module fills in form fields automatically from parameters in the URL request. We use it to automatically add the current route to the form when adding a new OPH item. While this is "safe" in that you have to be an admin to add a new OPH item and it only pre-fills some form fields, it feels like an insecure thing to do across the whole site.

We should find a way to pull the route automatically without the use of the Prepopulate module.

🐛 Bug report
Status

Needs review

Version

1.0

Component

Code

Created by

🇺🇸United States John Franklin

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @John Franklin
  • 🇦🇹Austria maxilein

    Cannot install using composer:

    composer require 'drupal/on_page_help:^1.0'
    ./composer.json has been updated
    Running composer update drupal/on_page_help
    
    Loading composer repositories with package information
    Updating dependencies
    Your requirements could not be resolved to an installable set of packages.
    
      Problem 1
        - Root composer.json requires drupal/on_page_help ^1.0 -> satisfiable by drupal/on_page_help[1.0.0].
        - drupal/on_page_help 1.0.0 requires composer/prepopulate ^2.4 -> could not be found in any version, there may be a typo in the package name.
    
    Potential causes:
     - A typo in the package name
     - The package is not available in a stable-enough version according to your minimum-stability setting
       see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.
     - It's a private package and you forgot to add a custom repository to find it
    
    Read <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.
    
    Installation failed, reverting ./composer.json and ./composer.lock to their original content.
    
    
    
    • e0fcd2b7 committed on 1.x
      Issue #3432085: Fix thinko in composer.json.
      
  • 🇺🇸United States John Franklin

    @maxilein: fixed in 1.0.1. Try it again, please.

  • First commit to issue fork.
  • Merge request !1Update file on_page_help.info.yml → (Open) created by immaculatexavier
  • Assigned to PrabuEla
  • Issue was unassigned.
  • Status changed to Needs review 6 months ago
  • 🇮🇳India PrabuEla chennai

    The dependency has been removed earlier. Just moving it to needs review

Production build 0.71.5 2024