• crypt
    link
    fedilink
    arrow-up
    0
    ·
    4 years ago

    оно сразу в разных местах. в частности ссылка на taglib (это критично?). поэтому я сейчас буду учиться задавать какой-то инклуд в jsp и инклудить оттуда variable. ты, если хочешь, можешь мне сразу показать короткий путь.

    # grep linux.org.ru ./src/main/webapp/WEB-INF/jsp/section-rss.jsp
    
    <%@ taglib prefix="l" uri="http://www.linux.org.ru" %>
        <title>Linux.org.ru: <c:out escapeXml="true" value="${ptitle}"/></title>
    
        <description>Linux.org.ru: <c:out escapeXml="true" value="${ptitle}"/></description>
    
    
    • cocucka
      link
      fedilink
      arrow-up
      0
      ·
      4 years ago

      taglib не надо трогать. А для текста надо прокидывать siteConfig в контекст jsp, но до этого надо вынести название сайта в конфиг.

      • crypt
        link
        fedilink
        arrow-up
        0
        ·
        4 years ago

        прям не надо или если очень хочется, то можно? как я написал в письме, оно замусоревает греп и мы теряем прочий хардкод. не только название сайта.

        • cocucka
          link
          fedilink
          arrow-up
          0
          ·
          4 years ago

          Не надо. Ты же не будешь package ru.org.linux.* тоже менять, а это одного поля ягоды

          • crypt
            link
            fedilink
            arrow-up
            0
            ·
            4 years ago

            я понял, что одного. но ru.org.linux не попадает в выдачу grep (основной инструмент непрограммиста). в любом случае есть смысл заменить имя этого неймспейса на что-то более нейтральное, не привязанное к имени сайта. иначе в этом просто нет логики: мы либо привязываемся к имени сайта и тогда должны его удобно менять через переменную, либо не привязываемся, тогда оно тем более не критично.

            ок, замению хардкодом, как скажешь, на что-то более нейтральное.

            • cocucka
              link
              fedilink
              arrow-up
              0
              ·
              4 years ago

              Неймспейс через переменную не заменить