Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
janadamec1 opened this issue Feb 20, 2020 · 2 comments

Comments

@janadamec1
Copy link

V commitu bcfbe22 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
Copy link
Member

jitka commented Feb 23, 2020

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

@janadamec1
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants