pirati-web / jekyll-theme-pirati

Web desing of Czech Pirate Party
6 stars 17 forks source link

Chybné generování obrázků když není zapnutý mediaStorage (kritická chyba v theme 7.7.0) #49

Open janadamec1 opened 4 years ago

janadamec1 commented 4 years ago

V commitu https://github.com/pirati-web/jekyll-theme-pirati/commit/bcfbe227ce774b5a9136f8dbbe66e8d336a7a2f8 je chybně zachytávání obrázků v podmínce na zapnutí mediaStorage. Pokud je zapnutý, zachytí to cestu, pokud vypnutý, zachytí celý tag . Díky tomu nefungují například profilové obrázky. Viz např https://praha12.pirati.cz/lide/matej-mlcoch/

Například right-bar/bar_foto.html se dá opravit následovně, ale chtělo by to opravit na všech místech stejným způsobem (a na to nemám kapacitu).

{% if page.img %}

{% if site.mediaStorage %}
    {% capture img_tag %} <img src="https://a.pirati.cz/resize/225x225/{{ site.mediaStorage }}/img/{{ page.img }}" alt="{{ page.name }}">{% endcapture %}
{%- else -%}
    {% capture img_tag %}{% asset '{{ page.img }}' alt='{{ page.name }}' magick:resize='350x' class='c-person-image' %}{% endcapture %}
{%- endif -%}

{% if page.profiles.flickr %}
    <div class="c-widget-accordion-content__cta"> </div>
    <div class="o-section-block">
      <ul class="c-widget-accordion" data-accordion="" data-allow-all-closed="true">
        <li class="c-widget-accordion-item is-active" data-accordion-item="">
          <div class="c-widget-accordion-content" data-tab-content="">
            {{ img_tag }}
            <div class="c-widget-accordion-content__cta">
              {% assign peopleCol = site.collections | where: "label", "people" | first %}
              <a class="expanded button large u-0margin" href="{{page.profiles.flickr}}"> Zobrazit více fotografií </a>
            </div>
          </div>
        </li>
      </ul>
    </div>
{% else %}
   {{ img_tag }}
{% endif %}

{% endif %}
jitka commented 4 years ago

ahoj, mělo by to být opravené, když tak na to koukni, ale ještě to čeká na novou verzi

janadamec1 commented 4 years ago

No v tom bar_foto je to asi opravené, ale podle mne je ta samá chyba i ve všechny ostatních souborech měněných v tom commitu.