I wrote a module to integrate RoundCube with Drupal.
There are already a few existing modules, but none for Drupal 7. As I migrated my Website (skateboard.de) to drupal 7 I was in need of such module :-)
The module is based on the logic of the Drupal 5 Module http://drupal.org/node/333333 I used before.
I added some more features:

  • Block to show unread messages from the imap server
  • Mailbox creation - you can add a script part, that will create mailboxes on the imap server when users log in or create a new account (samples for PLESK included)

I have the module online and running on my site since a week.

The project can be found here: Webmail3000.

The Source Code can be found here: git.drupal.org:sandbox/kingbbq/1536578.git

CommentFileSizeAuthor
#13 pareview.txt51.38 KBanwar_max

Comments

kingbbq’s picture

kingbbq commented

The integration with Roundcube is done via a roundcube plugin that is also part of the source.

this is the sample for PLESK with 50mb size of mailbox with Spamchecker and Virus-Checker on.
sudo /usr/local/psa/bin/mail -c $email -mailbox true -mbox_quota 50M -antivirus inout 2>&1;
sudo /usr/local/psa/bin/mail -u $email -passwd $password -mbox_quota 50M -antivirus inout 2>&1;
sudo /usr/local/psa/bin/spamassassin -u $email -status true -reject-spam true -hits 3 2>&1;

I will write detailed documentation on how to install in the next days...

patrickd’s picture

patrickd commented
Status: Needs review » Needs work

welcome,

You are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.

As installation and usage instructions are quite important for us to review, please take a moment to make your project page follow tips for a great project page. Also create a README.txt that follows the guidelines for in-project documentation.

while waiting for an in-depht review of your module you can start out fixing some coding style issues detected by automated tools:
http://ventral.org/pareview/httpgitdrupalorgsandboxkingbbq1536578git

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.

regards

kingbbq’s picture

kingbbq commented
Status: Needs work » Needs review

thanks patrick!

- I moved to a proper branch according to the documentation
- added readme.txt and documentation to the project page
- cleaned up errors from the automatic checking.

I will try to work on other reviews as recommended for the review bonus in the next days...

cheers
Gregor

patrickd’s picture

patrickd commented

Note that the branch you've created does not follow naming rules for beeing able to create releases.
Please read the branch naming conventions.

Also note that you can re-run the automated review yourself on the "repeat review"-tab.

kingbbq’s picture

kingbbq commented

thanks, changed it to 7.x - I hope that is the right branch name now.

cheers
Gregor

patrickd’s picture

patrickd commented

Well, now you created a core-named branch ;-)

Branch name Branch Description Downloadable release
7.x 7.x-dev release (core) drupal-7.x-dev.tar.gz,
drupal-7.x-dev.zip
-->7.x-1.x 7.x-1.x-dev release (contrib) myproject-7.x-1.x-dev.tar.gz,
myproject-7.x-1.x-dev.zip
7.x-2.x 7.x-2.x-dev release (contrib) myproject-7.x-2.x-dev.tar.gz,
myproject-7.x-2.x-dev.zip
kingbbq’s picture

kingbbq commented

g** damn it ;-)
Thanks for the hinting --> I am hoping for the best now :-)

luxpaparazzi’s picture

luxpaparazzi commented

shouldn't the module-name refer to RoundCube?
it would also be nice giving a small introduction do whatever RoundCube is...

kingbbq’s picture

kingbbq commented

All "normal" Module-Name like RoundCube or RoundCube Webmail are all already used (even though there is no more active project / no project for Drupal 7) so I choose this name. I provided a plugin for RoundCube, if other plugins for other webmails would be provided, the module would work also with others.

kingbbq’s picture

kingbbq commented

...Added explanation of RoundCube and screenshots.

lukas.fischer’s picture

lukas.fischer commented

Why don't you contact the the people who created http://drupal.org/project/rcmail or http://drupal.org/project/roundcube. I think it would make sense to work with existing projects. It's just my personal impression - What do others mean?

kingbbq’s picture

kingbbq commented

Hi Lukas,

sure, why not. I'll contact them... you mean, using me module under the existing RoundCube Module, just as a new version?

cheers
Gregor

anwar_max’s picture

anwar_max
Primary language English
commented
Status: Needs review » Needs work
StatusFileSize
new pareview.txt51.38 KB

There are still files other than README.txt in the master branch, make sure to remove them. See also step 5 in http://drupal.org/node/1127732
Review of the 7.x-1.x branch:

This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. Get a review bonus and we will come back to your application sooner.

Manual Review:

1) Use proper indention and proper line spacing in your code.
2) In webmail3000_form() function you are using placeholders in t() function t('@domain_name used to generate emails')
but you are not passing values for it.
3) Write documentation for each function.

progger’s picture

progger commented

I installed the "modul" It is good for the beginning. Thanks a lot. I saw that the login/logout, autologin, deletion, password change/update process is not working right now. Also the link version "click at webmail link to login roundcube" is sometimes not working correct with some browsers as the url is not correct somehow. I think anyway it would be better if the user has the chance to make a setting to login directly to roundcube intern area after drupal login.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.