Community
    • Categories
    • Recent
    • Popular
    • Users
    • Search
    • Register
    • Login

    Trouble Ticket System (TTS) config skips URL port

    Scheduled Pinned Locked Moved Development
    2 Posts 2 Posters 608 Views 1 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • bmsoftB Offline
      bmsoft
      last edited by

      Hello,

      We are using zammad at port 3000 and when we try to integrate this instance with i-doit. Investigating deeply about this problem we see that in constructor of class isys_connector_ticketing_zammad at $this->m_rt_base_url initialization is missed the port and i-doit won't make request to the correct url.

      We propose two ways to solve it:

      • Add port to rt base url:
          /**
           * Injects the used protocol and sets RT's base url.
           *
           * @param isys_protocol $p_protocol
           */
          public function __construct($p_protocol)
          {
              parent::__construct($p_protocol);
      
      
              $this->m_ticket_url_pattern = $this->m_protocol->get_base_url() . $this->m_ticket_url_pattern;
              $this->m_rt_base_url = $this->m_protocol->get_host() . ':' . $this->m_protocol->get_port() . $this->m_protocol->get_base_url();
              $this->m_protocol->attach_base_url($this->m_url_base);
          }
      
      • Change get_host method of class isys_protocol_http to add port to all implementations with this error (like all other connectors of TTS):
          /**
           * Returns the Host without any information
           */
          public function get_host()
          {
              return $this->m_protocol . "://" . $this->m_host . ':' . $this->m_port;
          }
      
      1 Reply Last reply Reply Quote 0
      • Philipp HörselmannP Offline
        Philipp Hörselmann
        last edited by

        Hey @bmsoft
        thank you for the hint.
        If this will be included in further version you can find this in the change logs.

        Best
        Phil

        1 Reply Last reply Reply Quote 0

        Hello! It looks like you're interested in this conversation, but you don't have an account yet.

        Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

        With your input, this post could be even better 💗

        Register Login
        • First post
          Last post