Contrib.social

D7: Move quant_api_seed_get logic and add update hook

Open on Drupal.org →
Created on 19 July 2023, almost 2 years ago

Problem/Motivation

When ✨ D7: Add option to allow seeding via GET or POST Fixed was added, the code was partially moved from the quant_api module to the main quant module, but uninstalling the variable wasn't moved. Also, the variable was still namespaced with quant_api. Also, it should default to TRUE rather than FALSE and, thus, needs an update hook.

Steps to reproduce

Proposed resolution

  1. Change variable from quant_api_seed_get to quant_seed_get
  2. Remove uninstall from quant_api.install and add to quant.install
  3. Default form field to TRUE
  4. Add update hook to default form value to TRUE

Remaining tasks

  • Update code
  • Review and test
  • Commit :)

User interface changes

Form field will default to true.

API changes

Data model changes

Variable will default to true.

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

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

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024