`OSPinserverPrivateStreamWrapper` is only added via 
hook_stream_wrappers_alter when the site is configured to use pin.  When 
this doesn't happen, the default DrupalPrivateStreamWrapper takes over.  
That's the one I was trying to get rid of.

On 10/28/2013 03:26 PM, Richard Brandon wrote:
>
> In 
> openscholar/modules/os_features/os_files_private/os_files_private.module:
>
> > @@ -220,8 +220,8 @@ function os_files_private_menu_local_tasks_alter(&$tasks, $router, $root) {
> >   *
> >   * @param array $wrappers
> >   */
> > -function os_files_private_stream_wrappers_alter(&$wrappers){
> > -  if (isset($wrappers['private'])) {
> > +function os_files_private_stream_wrappers_alter(&$wrappers){
> > +  if (isset($wrappers['private']) && $wrappers['private']['class'] != 'OSPinserverPrivateStreamWrapper') {
>
> All of our production installs are using 
> 'OSPinserverPrivateStreamWrapper' does this mean that your wrapper is 
> never used? If a user is using private files but not with the 
> pinserver aka(Extend to Harvard) isn't your code still needed?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/openscholar/openscholar/pull/4226/files#r7260160>.
>