• crypt
    0·
    4 years ago

    и еще такая фигня, что я не программист и хз, как лучше это легаси убирать из jsp. может, ты программист и знаешь, как лучше переменную в jsp засунуть?

    • cocucka
      0·
      4 years ago

      Это вряд ли вообще к jsp имеет отношение, ты имхо не туда смотришь.

      • crypt
        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
          0·
          4 years ago

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

          • crypt
            0·
            4 years ago

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

            • cocucka
              0·
              4 years ago

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

              • crypt
                0·
                4 years ago

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

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

                • cocucka
                  0·
                  4 years ago

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