8306569a-4f72-444d-ba8a-d69ed227106c

Committed 05 Nov 2025 07:43PM UTC coverage: 91.527% (+0.03%) from 91.5%

Build # 8306569a-4f72-444d-ba8a-d69ed227106c

Build Type

push

circleci

Committed by

web-flow

Commit Message

Set the displayed value to the entire name (#2151)

Centralize the view and javascript for data manager and sponsor
Set the displayed value to the entire name and have a hidden field with
the uid

fix spec error "not to search for nil"
<img width="1707" height="1085" alt="Screenshot 2025-11-05 at 2 38
43 PM"
src="https://github.com/user-attachments/assets/82278712-d16d-4b58-8b9c-5c33587a7810"
/>

Run Details

6 of 10 new or added lines in 1 file covered. (60.0%)

1010 existing lines in 37 files now uncovered.

2841 of 3104 relevant lines covered (91.53%)

792.96 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

97.67

/app/models/mediaflux/query_request.rb

# frozen_string_literal: true
module Mediaflux
  class QueryRequest < Request
    attr_reader :aql_query, :collection, :action, :deep_search, :iterator

    # Constructor
    # @param session_token [String] the API token for the authenticated session
    # @param aql_query [String] Optional AQL query string
    # @param collection [Integer] Optional collection id
    # @param action [String] Optional, by default it uses get-name but it could also be get-meta to get all
    #                        the fields for the assets or `get-values` to get a limited list of fields.
    # @param deep_search [Bool] Optional, false by default. When true queries the collection and it subcollections.
    # @param iterator [Bool] Optional, true by default. When true returns an iterator.  When false returns a list of results
    def initialize(session_token:, aql_query: nil, collection: nil, action: "get-values", deep_search: false, iterator: true)
      super(session_token: session_token)
      @aql_query = aql_query
      @collection = collection
      @action = action
      @deep_search = deep_search
      @iterator = iterator
    end

    # Specifies the Mediaflux service to use when running a query
    # @return [String]
    def self.service
      "asset.query"
    end

    # Returns the iterator that could be used to fetch the data
    def result
      xml = response_xml
      xml.xpath("/response/reply/result/iterator").text.to_i
    end

    def result_items
      xml = response_xml
      xml.xpath("/response/reply/result").children.map do |node|
        {
          id: node.xpath("./@id").text,
          name: node.xpath("./name").text,
          path: node.xpath("./path").text
        }
      end
    end

    private

      def build_http_request_body(name:)
        super do |xml|
          xml.args do
            # TODO: there is a bug in mediaflux that does not allow the comented out line to paginate
            #      For the moment we will utilize the where clasue that does allow pagination
            # xml.collection collection if collection.present?
            if collection.present?
              xml.where mf_where(collection)
            end
            xml.where aql_query if aql_query.present?
            xml.action action if action.present?
            declare_get_values_fields(xml) if action == "get-values"
            xml.as "iterator" if iterator
          end
        end
      end

      def mf_where(collection)
        if deep_search
          "asset in static collection or subcollection of #{collection}"
        else
          "asset in collection #{collection}"
        end
      end

      # Adds the declarations to fetch specific fields
      def declare_get_values_fields(xml)
        declare_get_value_field(xml, "name", "name")
        declare_get_value_field(xml, "path", "path")
        declare_get_value_field(xml, "content/@total-size", "total-size")
        declare_get_value_field(xml, "mtime", "mtime")
        declare_get_value_field(xml, "@collection", "collection")
      end

      # Adds a single field declaration
      def declare_get_value_field(xml, field_xpath, field_name)
        xml.xpath do
          xml.parent.set_attribute("ename", field_name)
          xml.text(field_xpath)
        end
      end
  end
end

1	# frozen_string_literal: true
2	module Mediaflux	5✔
3	class QueryRequest < Request	5✔
4	attr_reader :aql_query, :collection, :action, :deep_search, :iterator	5✔
5
6	# Constructor
7	# @param session_token [String] the API token for the authenticated session
8	# @param aql_query [String] Optional AQL query string
9	# @param collection [Integer] Optional collection id
10	# @param action [String] Optional, by default it uses get-name but it could also be get-meta to get all
11	# the fields for the assets or `get-values` to get a limited list of fields.
12	# @param deep_search [Bool] Optional, false by default. When true queries the collection and it subcollections.
13	# @param iterator [Bool] Optional, true by default. When true returns an iterator. When false returns a list of results
14	def initialize(session_token:, aql_query: nil, collection: nil, action: "get-values", deep_search: false, iterator: true)	5✔
15	super(session_token: session_token)	91✔
16	@aql_query = aql_query	91✔
17	@collection = collection	91✔
18	@action = action	91✔
19	@deep_search = deep_search	91✔
20	@iterator = iterator	91✔
21	end
22
23	# Specifies the Mediaflux service to use when running a query
24	# @return [String]
25	def self.service	5✔
26	"asset.query"	182✔
27	end
28
29	# Returns the iterator that could be used to fetch the data
30	def result	5✔
31	xml = response_xml	67✔
32	xml.xpath("/response/reply/result/iterator").text.to_i	67✔
33	end
34
35	def result_items	5✔
UNCOV 36	xml = response_xml	22✔
UNCOV 37	xml.xpath("/response/reply/result").children.map do \|node\|	22✔
38	{
UNCOV 39	id: node.xpath("./@id").text,	25✔
40	name: node.xpath("./name").text,
41	path: node.xpath("./path").text
42	}
43	end
44	end
45
46	private	5✔
47
48	def build_http_request_body(name:)	5✔
49	super do \|xml\|	182✔
50	xml.args do	182✔
51	# TODO: there is a bug in mediaflux that does not allow the comented out line to paginate
52	# For the moment we will utilize the where clasue that does allow pagination
53	# xml.collection collection if collection.present?
54	if collection.present?	182✔
55	xml.where mf_where(collection)	118✔
56	end
57	xml.where aql_query if aql_query.present?	182✔
58	xml.action action if action.present?	182✔
59	declare_get_values_fields(xml) if action == "get-values"	182✔
60	xml.as "iterator" if iterator	182✔
61	end
62	end
63	end
64
65	def mf_where(collection)	5✔
66	if deep_search	118✔
67	"asset in static collection or subcollection of #{collection}"	118✔
68	else
69	"asset in collection #{collection}"	×
70	end
71	end
72
73	# Adds the declarations to fetch specific fields
74	def declare_get_values_fields(xml)	5✔
75	declare_get_value_field(xml, "name", "name")	178✔
76	declare_get_value_field(xml, "path", "path")	178✔
77	declare_get_value_field(xml, "content/@total-size", "total-size")	178✔
78	declare_get_value_field(xml, "mtime", "mtime")	178✔
79	declare_get_value_field(xml, "@collection", "collection")	178✔
80	end
81
82	# Adds a single field declaration
83	def declare_get_value_field(xml, field_xpath, field_name)	5✔
84	xml.xpath do	890✔
85	xml.parent.set_attribute("ename", field_name)	890✔
86	xml.text(field_xpath)	890✔
87	end
88	end
89	end
90	end

pulibrary / tigerdata-app / 8306569a-4f72-444d-ba8a-d69ed227106c

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous