Implementation of hook_requirements(), warning on configuration path is still present.

CommentFileSizeAuthor
persistent_login.hook_.requirements.patch2.61 KBpawel_r
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gapple’s picture

gapple
he/they
Primary language English
CreditAttribution: gapple commented
Title: Implementation of hook_requirements() patch » Display session.cookie_lifetime configuration message via hook_requirements()
Version: 6.x-1.4 » 6.x-1.x-dev
Category: feature » task
Status: Patch (to be ported) » Needs review

Thank you for the patch, things look good on a quick review.

pawel_r’s picture

pawel_r CreditAttribution: pawel_r commented

No problem. On the second look, description for status "Configured" could be omitted.

gapple’s picture

gapple
he/they
Primary language English
CreditAttribution: gapple commented
Priority: Normal » Minor
Status: Needs review » Fixed

Fixed in commits:

fb4be1dd822a886e4fcbfde02038484345db3ffd (6.x)

e277b26edca7ab2850878542571c5829a7215d3f (7.x)

Status: Fixed » Closed (fixed)

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