We have run into a problem with some hosting environments(Pantheon) and have built this tiny module to provide a database storage for the files. It uses "Private Files Storage" under the hood, but transparently synchronizes it with a database table. It is useful in cases where you have an issue with file-based wrapper (such as Temporary, Private and Public) and need the files to be consistent or you don't want to give someone access to the files storage (or can't do it because of technical reasons), but it's fine to give a table access just for some files storage.

Project URL: https://www.drupal.org/sandbox/jim/2352733
GIT: http://git.drupal.org/sandbox/Jim/2352733.git database_stream_wrapper

Comments

Jim.M’s picture

Issue summary: View changes
PA robot’s picture

Status: Active » Needs work

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

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.

Jim.M’s picture

Status: Needs work » Needs review

We've fixed the necessary code issues from the report, some of the "issues" aren't fixed because of no sense of them - for instance, "Public method name "DatabaseStreamWrapper::stream_open" is not in lowerCamel format" - it's an abstract method overriding, so I can't change the name.

Thanks,
Jim

sajiniantony’s picture

Status: Needs review » Needs work

Hello,
Readme.txt is missing in the module folder.All contributed modules should provide a README.txt or README.md file in the package.
It would be nice to add hook_help() as this is a short module because it only contain 2 functions and only 37 lines of code.

Jim.M’s picture

Status: Needs work » Needs review

I've commited and pushed README and hook_help implementation to the module repo.

InviteReferrals’s picture

Status: Needs review » Needs work
Issue tags: +Errors in file DatabaseStreamWrapper.inc

FILE: /var/www/drupal-7-pareview/pareview_temp/DatabaseStreamWrapper.inc
---------------------------------------------------------------------------
FOUND 5 ERRORS AND 1 WARNING AFFECTING 6 LINES
---------------------------------------------------------------------------
7 | WARNING | [ ] Line exceeds 80 characters; contains 84 characters
18 | ERROR | [x] Doc comment long description must start with a capital
| | letter
20 | ERROR | [ ] Public method name
| | "DatabaseStreamWrapper::stream_open" is not in
| | lowerCamel format
39 | ERROR | [x] Doc comment long description must start with a capital
| | letter
41 | ERROR | [ ] Public method name
| | "DatabaseStreamWrapper::stream_close" is not in
| | lowerCamel format
69 | ERROR | [x] The closing brace for the class must have an empty
| | line before it

klausi’s picture

Status: Needs work » Needs review
Issue tags: -Errors in file DatabaseStreamWrapper.inc

Those are surely not application blockers, please do a real manual review.

Jim.M’s picture

Klausi, Thanks.

It seems i've previously already been approved to post full modules but forwhatever reason now i'm not.

klausi’s picture

Status: Needs review » Closed (duplicate)
Already Approved
It appears that you have successfully completed the application process with a different application, and been granted the 'create full projects' permission.

Once their first application has been successfully approved, then an applicant can promote other projects without review. Because of this, posting multiple applications are not necessary, and results in additional workload for reviewers!

Instead, you can post to the Peer Review group for feedback, here:

https://groups.drupal.org/peer-review/about

With this in mind, I have marked your application as 'closed(duplicate)'. If this is incorrect, and you do not yet have the ability to create full projects, then please feel free to open an issue in the webmaster queue https://www.drupal.org/project/issues/webmasters

apaderno’s picture

Related issues: +#1163318: [D7] Domain Feeds