devel inline is a module that empersize the execute code form of devel module. It's give you option to see all of the available classes, interfaces and hooks. Why hooks? you can defined a temporary hooks for debugging the hooks interaction in your site with out edit the files. This is also good for debugging the hooks on the live site.

Made for drupal 7.
Clone:
git clone --branch 7.x-1.x RoySegall@git.drupal.org:sandbox/RoySegall/1645754.git

Sandbox: http://drupal.org/sandbox/RoySegall/1645754

Comments

brazorf’s picture

brazorf commented
Status: Needs review » Needs work

Hello there,

1. you should remove LICENSE.txt as it will be automatically added by drupal packager
2. run pareview, there are a couple of things to be fixed there

Regards,
brazorf

roysegall’s picture

roysegall commented
Status: Needs work » Needs review

fixes applyed, checked using code review.

mitchell’s picture

mitchell commented
Status: Needs review » Needs work

Thank you for contributing this Roy! It looks very interesting. Installation went fine, but I'm having trouble using it. Maybe write up a step by step guide on how to do something in particular with the module. If you want some help with the copy review or translation, I can help with that if you'd like.

roysegall’s picture

roysegall commented

There is no a "step by step". I wrote it in the readme file - all you need to do is to start write a code in the dvevl php execute code form and then you get an auto complete of the functions - just like a normal IDE.

Another thing i'd like to mention - this module is still beta and i whant to get out from sandbox so more people code download it easily and help make it a better one. I think there is a great potential in this module.

rho_’s picture

rho_ commented

Passes pareview, code is clean and legible, installs cleanly, and works as described.

It does seem that it's fairly easy to confuse the autocomplete, but this seems to be noted in the README, and covered by "beta."

I did a bit of googling to see if something like this exists already, and was unable to find anything similar.

Overall, good job! REALLY cool module. I would love to see this developed further.

rho_’s picture

rho_ commented
Status: Needs work » Needs review

Meant to switch back to needs review in my last post. whoops

roysegall’s picture

roysegall commented

Still need work on the reademe file.

roysegall’s picture

roysegall commented

OK, i made a better read me file and write it down in the module(It's an AWSOME read me file).

I think this module can go from sandbox to an early beta module for feed backs and suggestion by other users.

roysegall’s picture

roysegall commented

Someone?

roysegall’s picture

roysegall commented

Please guys - can some one give me a response?

patrickd’s picture

patrickd commented

We do really need more hands in the application queue and highly recommend to get a review bonus so we can come back to your application sooner.

Sorry

vaibhavjain’s picture

vaibhavjain
he/him
Primary language Hindi
Location Delhi
commented
Status: Needs review » Needs work

Firstly , you link to GIT is not correct, should be - http://git.drupal.org/sandbox/RoySegall/1645754.git
Results from automated review - http://ventral.org/pareview/httpgitdrupalorgsandboxroysegall1645754git

roysegall’s picture

roysegall commented

I applied the two fixes.

patrickd’s picture

patrickd commented
Status: Needs work » Needs review

don't forget to switch back to needs review

mitchell’s picture

mitchell commented
Status: Needs review » Reviewed & tested by the community

IMHO, this one is ready. I tested the module, scanned the code, and see that ventral doesn't recommend anything else.

I don't see any minimum recommended time in the workflow for the RTBC state (self-reminder: todo), but I would prefer to complete this one soon. I think 12hrs or 24hrs is reasonable, so @other-reviewers, please confirm or make another state change before then if you have a few min.

roysegall’s picture

roysegall commented

@mitchell, i didn't understood - does the module ok?

mitchell’s picture

mitchell commented
Title: Inline devel » Inline Devel

One last issue (normal priority, so no hold up), #1699210: The project's name should be uppercased and canonical.

mitchell’s picture

mitchell commented

@RoySegall: Thanks for fixing #17.

> @mitchell, i didn't understood - does the module ok?
I'll translate for you.. and also for the fun of it.. I very rarely write in Hebrew, so your corrections would be appreciated :) ..רק הייתי בעולפן ב
אני חושב שזה מוחן. בדקתי המדול, קריאתי הקוד, ורואה שוינתראל לא הפקיד אוד מהשהו.

אני לא רואה שיש זמן מומלץ קטן ביותר בהטקסי..
rtbc (מהומה בישבילי),
אבל אני מעדיף להשלים אלה בקרוב. חושב 12ש"ע או 24ש"ע מספיק, אז @אמבקרים אחרים, בבקשה תאש ותעשה אוד שינוי להטקסי ליפני זה אמ יש לחה שנייה.

My next follow should come shortly, granting access. :)

mitchell’s picture

mitchell commented
Status: Reviewed & tested by the community » Fixed

Thanks again for your contribution, RoySegall!

I updated your account to let you promote this to a full project and also create new projects as either a sandbox or a "full" project.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on irc in #drupal-contribute and #drush. So, come hang out and stay involved! Thanks again and good luck! :)

roysegall’s picture

roysegall commented
Status: Fixed » Closed (fixed)

I'll fix the typos:

הייתי באולפן ב....
אני חושב שזה מוכן. בדקתי את המודול, קראתי את הקוד, ואני רואה שוינתראל לא הפיק משהו מיוחד.

אני לא רואה שיש זמן מומלץ מינמאלי אבל אני מעדיף להשלים את זה בקרוב.

אני חושב ש12-24 שעות זה מספיק, אז מבקרים אחרים, אנא תאשרו ותיישמו שינויים.

Thank you very much mitchel. i hope this module will improve. As i said, he have great potential.

mitchell’s picture

mitchell commented

#20: Very cool. Thanks for the corrections.

Best wishes.

mitchell’s picture

mitchell commented
Issue summary: View changes

Minor fixes.