1) This modules provides shortcode option to execute php scripts inside blocks and nodes. In Drupal 8, there is no way of using php filter but users might need it in worst cases. Keeping this in mind, we have created this module. It depends on "Shortcode" module.

2) Sandbox project page - https://www.drupal.org/sandbox/niyas/2780793

3) git clone --branch 8.x-1.x https://git.drupal.org/sandbox/Niyas/2780793.git php_shortcode

Comments

Niyas created an issue. See original summary.

PA robot’s picture

Issue summary: View changes

Fixed the git clone URL in the issue summary for non-maintainer users.

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

dilipsingh02’s picture

Status: Needs review » Needs work

Hi,

Please fixed the following error :

FILE: /var/www/drupal-7-pareview/pareview_temp/php_shortcode.info
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 1 LINE
----------------------------------------------------------------------
1 | ERROR | "description" property is missing in the info file
1 | ERROR | "core" property is missing in the info file
----------------------------------------------------------------------
niyas’s picture

@dilipsingh02

Is this information for .info file really needed being its Drupal 8 project? Anyway I added it.

Kindly check it :)

niyas’s picture

Status: Needs work » Needs review
dilipsingh02’s picture

Status: Needs review » Needs work

Hi Niyas,

please set git Default branch on your project page.
https://www.drupal.org/node/2780793/edit/default-branch

Thanks

klausi’s picture

Status: Needs work » Needs review

Setting the git default branch alone is surely not an application blocker. Anything else that you found or should this be RTBC instead?

dilipsingh02’s picture

Hi,

I have try to clone the module using given command but only .info file is download.
git clone --branch 7.x-1.x https://git.drupal.org/sandbox/Niyas/2780793.git

please check files and update the git.

Thanks

arun ak’s picture

Status: Needs review » Needs work

Files missing in git repository. Please commit files properly. For further information about how to manage project with git refer Maintaining a drupal.org project with Git.

joachim’s picture

What's the difference between this and https://www.drupal.org/project/php ?

niyas’s picture

Sorry guys, I added all missed files. Kindly check it and correct me if I am wrong.

@joachim - I think php shortcode can be used to execute php scripts in any filter format enabled field instead of creating standalone php filter.

kamdanishit’s picture

joachim’s picture

> @joachim - I think php shortcode can be used to execute php scripts in any filter format enabled field instead of creating standalone php filter.

The PHP module adds a filter which you can use in any text format. Aren't you saying the same thing about PHP Shortcode?

niyas’s picture

Issue summary: View changes
Status: Needs work » Needs review

@kamdanishit Thanks for your comments. I fixed it. Please check.

@joachim If we use php filter in any wysiwyg editor enabled fields, it creates an issue since it breaks the php code even though we enabled php evaluator. It might work for the first time, but if you edit the field again and see the code, "<?php" replaced by " <!--?php"

I hope it can be avoided using php_shortcode module since the shortcode can be used regardless of plain html field or editor enabled.

Please take a look at the screenshot http://prnt.sc/c6uecn

PA robot’s picture

Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpsgitdrupalorgsandboxNiyas2780793git

I'm a robot and this is an automated message from Project Applications Scraper.

joachim’s picture

That problem with wysiwyg sounds like a bug in the Php module, rather than something that requires a new separate project. Have you filed an issue for it?

PA robot’s picture

Status: Needs work » Closed (won't fix)

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.