CCK Authour module will create a cck filed in your content type , cck author will show who had created the node or who is the author of the content .

It's a simple module to configure and use .

Project Link
https://www.drupal.org/sandbox/surendra77c/2703111

Projects Git URL
git clone --branch 7.x-1.x https://git.drupal.org/sandbox/surendra77c/2703111.git cck_author

PAreview
http://pareview.sh/pareview/httpgitdrupalorgsandboxsurendra77c2703111git

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

surendra77c created an issue. See original summary.

PA robot’s picture

Status: Needs review » Needs work

Git clone command for the sandbox is missing in the issue summary, please add it.

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.

dkolarevic’s picture

Hi surendra77c,

Please check http://pareview.sh/pareview/httpgitdrupalorgsandboxsurendra77c2703111git and try to fix errors.

Manual review:
Lines 44 and 53 - you declared $settings variable but never used it.
Lines 114-115 - you can combine those nested ifs in one if statement.

surendra77c’s picture

Status: Needs work » Needs review

Requested changes compleated .

surendra77c’s picture

Status: Needs review » Active

Hi Team can I get the update on the module when it will be tested .

mike.davis’s picture

Hi @surendra77c, could you please update the issue with the URL of your sandbox module, as this will make it easier to review.

mike.davis’s picture

Having had a look at the Pareview report there is still a lot of code standard errors which will need to be fixed.

I have managed to find your sandbox module, and having had a look through the code, there is a print_r in the hook_node_presave function.

function cck_author_node_presave($node) {
print_r($node);
}

Please remove this as this function doesn't look like it is needed and there shouldn't be a print_r in any production code.

mike.davis’s picture

Status: Active » Needs review

Updated the status

mike.davis’s picture

Status: Needs review » Needs work
surendra77c’s picture

Status: Needs work » Needs review

Hi Team ,

Requested changes completed can you please review the code .

surendra77c’s picture

Status: Needs review » Active

Hi Team ,

Can you please review the code .

mike.davis’s picture

Status: Active » Needs work

Hi @surendra77c,

Although you have removed the print_r, the coding standards raised in the Pareview report are still present.

You should look at fixing these first so that this report passes before continuing.

Thanks
Mike

immanuel.paul’s picture

Hi surendra77c,

I have created a patch for your module based on pareview.sh report. Please review the patch and apply to your code. Good luck :)

Thanks
Immanuel Paul

surendra77c’s picture

Status: Needs work » Active

Hi Team ,

Thanks @ Immanuel Paul for your patch , that has help me to fix my module issue . I have update the code can you please review the code .

surendra77c’s picture

Status: Active » Needs review

Hi Team ,

Can you please review the code .

surendra77c’s picture

Status: Needs review » Active

Hi Team ,

Can you please review the code .

gisle’s picture

Status: Active » Needs review

To get people to review your application, you need to set its state to "Needs review", not "Active". Please see project applicaton workflow.

If you're impatient, to give your application a higher priority, please join the review bonus system. If you participate in the system by doing manual reviews, you can put yourself on the high priority list. Applications with PAReview: bonus tag are more likely to receive attention from reviewers and git administrators.

ashwinsh’s picture

Issue summary: View changes

Added Project link and Git URL for this poject.

surendra77c’s picture

Hi gisle ,

I have set the project to review state , can you please review the code now .

surendra77c’s picture

.

gisle’s picture

Issue summary: View changes

Your code isn't clean enough yet for me to bother with doing a code review. While you're waiting for one, you can work on the following:

Fix the coding style errors pointed out by http://pareview.sh/pareview/httpgitdrupalorgsandboxsurendra77c2703111git (style errors don't block promotion, but it is more difficult and more work to manually review code with style errors, so I usually don't do a manually review until the code is clean).

Remove the patch that is included in your your repo, it doesn't belong there.

Your project page is empty. What is the use case for this module? As far as I am able to tell, it creates a text field to hold an author name. That field that can be added to the content type under "manage fields". Adding a text field to a content type is already in core. What utility does your module provide that is not already in the core fields module? Please take a moment to make your project page follow the Project page template and it may be a good idea to also read tips for a great project page.

Set the correct values for maintenance status and development status.

Make your README.txt follow the guidelines for in-project documentation and the README.txt Template.

PS: Please also note that there are more applications that yours that Needs review. Some have been waiting longer than yours.

chaitanya17’s picture

Hi @surendra77c,

I have reviewed your module, nice to see author field to content type created.

You have written hook_schema($field) { //logic } in cck_author.module file.
Proper way will be using .install file. please move related code to install file.

Thanks.

chaitanya17’s picture

Status: Needs review » Needs work
pookmish’s picture

Manual Review

Individual user account
Yes: Follows the guidelines for individual user accounts.
No duplication
Yes: Does not cause module duplication and/or fragmentation.
Master Branch
Yes: Follows the guidelines for master branch.
Licensing
Yes: Follows the licensing requirements.
3rd party assets/code
Yes: Follows the guidelines for 3rd party assets/code.
README.txt/README.md
Yes: Follows the guidelines for in-project documentation and/or the README Template.
Code long/complex enough for review
Yes: Follows the guidelines for project length and complexity.
Secure code
Yes: Meets the security requirements.
Coding style & Drupal API usage

  • Why display the field label if nothing can be altered
  • hook_field_schema should reside in a .install file
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.

surendra77c’s picture

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

Hi ,

I have moved schema code to .install file please review the code .

surendra77c’s picture

Status: Needs review » Active

Team please review the code .

surendra77c’s picture

Status: Active » Needs review
Satyam Upadhyay’s picture

Hi surendra77c,

I reviewed your module, but it's not clear what you want to implement using this module in cck fields and nothing to be add from the patch that is added in the clone folder.

And you can see nothing happened using this module http://www.screencast.com/t/HKXRD1uYVb6
I think that you want to implement, this can be achieve by using core cck_field by added a text field in to any ctype

Regards
Satyam

Satyam Upadhyay’s picture

Status: Needs review » Needs work
Satyam Upadhyay’s picture

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.