In 99% of sheetnode installations, the paths to required libraries will be precisely located as described in the setup instructions. Therefore it would be prudent to try using those paths when no configuration is set. The supplied patch makes it so.

And as a bonus, this change fixes bug #1867286 (Blank page when trying to save PHPExcel and TCPDF path settings). It also adds support for a <drupalroot> token in user-specified library paths--this is important for keeping self-contained Drupal installations environment-agnostic, and is exactly the sort of thing that would make a working dev site break when deploying to staging/production.

CommentFileSizeAuthor
#1 autopath.diff5.12 KBHonoredMule
autopath.diff5.13 KBHonoredMule
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

HonoredMule’s picture

FileSize
5.12 KB

Tiny change...this time we're not introducing a redundant directory separator in the default paths.

HonoredMule’s picture

Issue summary: View changes

using html entities

infojunkie’s picture

Issue summary: View changes
Status: Needs review » Fixed

Thanks for you patch. I committed similar code to the latest dev.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.