Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct the documentation comment for field_config_update_validate() #6319

Closed
kiamlaluno opened this issue Dec 3, 2023 · 3 comments · Fixed by backdrop/backdrop#4593
Closed

Comments

@kiamlaluno
Copy link
Member

kiamlaluno commented Dec 3, 2023

In the field.module file, field_config_update_validate() has the following documentation comment.

/**
 * Implements hook_config_create_update().
 */

Since hook_config_create_update() is not a documented hook, field_config_update_validate() must be a hook_config_update_validate() implementation.

@kiamlaluno kiamlaluno changed the title field_config_update_validate() is defined twice field_config_update_validate() has the wrong documentation comment Dec 3, 2023
@kiamlaluno
Copy link
Member Author

(I apologize: At 8:30 PM, I probably had hallucinations when I was watching the code.)

@argiepiano
Copy link

RTBC

@kiamlaluno kiamlaluno changed the title field_config_update_validate() has the wrong documentation comment Correct the documentation comment for field_config_update_validate() Dec 24, 2023
@quicksketch
Copy link
Member

Thanks @kiamlaluno and @argiepiano! Merged backdrop/backdrop#4593 into 1.x and 1.26.x.

@quicksketch quicksketch added this to the 1.26.4 milestone Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants