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
Comment #1
Jim.M CreditAttribution: Jim.M commentedComment #2
PA robot CreditAttribution: PA robot commentedThere 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.
Comment #3
Jim.M CreditAttribution: Jim.M commentedWe'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
Comment #4
sajiniantony CreditAttribution: sajiniantony commentedHello,
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.
Comment #5
Jim.M CreditAttribution: Jim.M commentedI've commited and pushed README and hook_help implementation to the module repo.
Comment #6
InviteReferrals CreditAttribution: InviteReferrals commentedFILE: /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
Comment #7
klausiThose are surely not application blockers, please do a real manual review.
Comment #8
Jim.M CreditAttribution: Jim.M commentedKlausi, Thanks.
It seems i've previously already been approved to post full modules but forwhatever reason now i'm not.
Comment #9
klausiOnce 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
Comment #10
apaderno