This module allows editing the site code for development. The code change is immediate and can be reverted there as well. The code update is done using sftp.

Project link

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

Git instructions

git clone --branch 8.x-1.x https://git.drupal.org/project/sceditor.git

PAReview checklist

https://pareview.sh/pareview/https-git.drupal.org-project-sceditor.git

CommentFileSizeAuthor
#6 Error-3025526.png117.06 KBmanish34jain

Comments

abhaisasidharan created an issue. See original summary.

avpaderno’s picture

Issue summary: View changes

Thank you for applying! Remember to change status when the project is ready to be reviewed.

avpaderno’s picture

Status: Active » Closed (won't fix)

If you are still working on this application, you should fix all known problems and set the status to Needs review. (See also the project application workflow.)
Please don't change status of this application if you aren't sure you have time to dedicate to this application, or it will be closed again as won't fix.

I am closing this application due to lack of activity.

abh.ai’s picture

Sorry about that. Was busy. I'm changing the status to needs review. 8.x-1.4 is working fine.

abh.ai’s picture

Status: Closed (won't fix) » Needs review
manish34jain’s picture

StatusFileSize
new117.06 KB

Hello Abhaisasidharan,

I installed this on https://simplytest.me/. And i am getting website encountered an unexpected error message.

Please check the attached screenshot of error log.

Thanks

abh.ai’s picture

Assigned: Unassigned » abh.ai
Status: Needs review » Active
avpaderno’s picture

Assigned: abh.ai » Unassigned
Status: Active » Needs review
saesa’s picture

Title: Source Code Editor » [D8] Source Code Editor
valentine94’s picture

Status: Needs review » Needs work

Hello, checked your module's code. Have the following advice to make it better:

In SCEditorController::scEditorPage you have to prevent using $_GET or $_POST superglobal variables it potentially dangerous and can cause the security issues, use Drupal's built-in request service instead, which can be called using DI as $container->get('request_stack')->getCurrentRequest() or as static method \Drupal::request().

Also, I don't see a LICENSE file within your module.

Regards, Valentine.

abh.ai’s picture

Thanks Valentine94. I am updating now.

saesa’s picture

Issue summary: View changes

Added PAReview checklist to summary

avpaderno’s picture

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

If you are still working on this application, you should fix all known problems and set the status to Needs review. (See also the project application workflow.)
Please don't change status of this application if you aren't sure you have time to dedicate to this application, or it will be closed again as won't fix.