Пути к статике - погибче бы настройку #54

Closed
puzrin opened this Issue Jan 16, 2013 · 6 comments

Comments

Projects
None yet
2 participants

puzrin commented Jan 16, 2013

Посмотрел по документации и коду, если я правильно понял, то пути к статике (flash) почти приколочены гвоздями - можно указать папку, а дальше клеятся константы.

Но есть проекты, где используются генераторы ассетов типа sprokets (ruby) и mincer (node). Там к именам клеятся md5/sha1, и на этом построено кеширование. Т.к. вспомогательных файлов у вашего модуля очень мало, хотелось бы иметь возможность конфигурировать их имена напрямую. Иначе понадобится либо nginx костылировать, либо сидеть без кеша на флешовых файлах.

Кратко - я хотел бы сам задавать имя каждого вспомогательного файла (флешового), без автоматической магии.

Collaborator

RubaXa commented Jan 16, 2013

Не вопрос, сегодня сделаю.

RubaXa added a commit that referenced this issue Jan 16, 2013

Merge pull request #55 from RubaXa/master
#54: added `FileAPI.flashUrl` and `FileAPI.flashImageUrl`

puzrin commented Jan 16, 2013

Thanks!

  1. Там написано, что staticPath required. Это все еще так или просто неточность?
  2. https://github.com/mailru/FileAPI/blob/master/lib/FileAPI.Flash.js#L80 - все-таки там осталась магия по добавлению версии API. Зачем делалось - понимаю. Насчет правильности в любом случае - не уверен. Мне вроде мешать не должно.
Collaborator

RubaXa commented Jan 16, 2013

  1. Не совсем точный коммент, имелось ввиду или + все же можно определить только одну опцию, а другую нет.
  2. Увы, но лучшего решения не могу предложить.

puzrin commented Jan 16, 2013

Ок, (2) мне ничего не сломает, не вижу смысла дополировывать.

(1) Если опции взаимоисключающие, может тогда использовать одну переменную и проверять тип? Когда строка - подкаталог. Когда объект - набор путей для каждого файла. IMHO, будет понятнее и не должно прибавить ошибок.

Collaborator

RubaXa commented Jan 16, 2013

Согласен, изначально даже написал @deprecated, но в последний момент передумал и сделал так :]

@RubaXa RubaXa closed this Jan 16, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment