The module provides a page like the "Single export" page for getting configuration in PHP array style.

Project link

https://www.drupal.org/project/config2php

Git instructions

git clone --branch 2.x https://git.drupalcode.org/project/config2php.git

PAReview checklist

https://pareview.sh/pareview/https-git.drupal.org-project-config2php-2.x

Comments

chmez created an issue. See original summary.

shaktik’s picture

Issue summary: View changes

Update git URL.

avpaderno’s picture

Issue summary: View changes

Thank you for applying! I added the Git instructions for non-maintainer users. Reviewers will check the project and post comments to list what should be changed.

avpaderno’s picture

klausi’s picture

Status: Needs review » Fixed
  1. I don't understand the use case for this module, why do you need the config as PHP. Please add that to the project page.
  2. I checked the module if there is a security issue with importing and executing config PHP, but it does only do the export so this should be safe.
  3. The regular expression rewriting could fail with special config field values that contain "false," for example. Don't have a better idea, but there could be some PHP parsing library for rewriting this in a more robust way.

Otherwise looks good to me.

Thanks for your contribution, Oleksandr!

I updated your account so you can opt into security advisory coverage now.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on Slack or IRC in #drupal-contribute. So, come hang out and stay involved!

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

Thanks to the dedicated reviewer(s) as well.

ohorbatiuk’s picture

Thank you @shaktik @kiamlaluno @klausi for review and your invaluable suggestions.

Status: Fixed » Closed (fixed)

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