weko#24731 - fixbug when click "Information" of manual file

Author: weko3-dev040

modules/weko-deposit/weko_deposit/api.py

@@ -1340,8 +1340,9 @@ def display_file_info(self):
                 exclude_attr = [
                     'displaytype', 'accessrole', 'licensetype', 'licensefree']
                 filename = request.args.get("filename", None)
-                for f in file_metadata:
-                    if f.get('filename', None) == filename:
+                file_order = int(request.args.get("file_order", -1))
+                for idx, f in enumerate(file_metadata):
+                    if file_order == idx or f.get('filename') == filename:
                         # Exclude attributes which is not use.
                         for ea in exclude_attr:
                             if f.get(ea, None):

modules/weko-records-ui/weko_records_ui/templates/weko_records_ui/body_contents.html

@@ -94,7 +94,7 @@ <h3 class="panel-title">{{_('Item')}}</h3>
 
                   {%- set access_permission = record | check_file_permission(file) -%}
                   {%- set file_details_url = url_for('invenio_records_ui.recid_file_details', pid_value=pid.pid_value,
-                      filename=file.filename) %}
+                      filename=file.filename or '[No FileName]') %}
 
                   <tr class="file-row">
                     <td rowspan="1" class="filecheck-td">
@@ -287,7 +287,11 @@ <h3 class="panel-title">{{_('Item')}}</h3>
                         {%- endif -%}
                       {%- endif -%}
                       <!-- Show button 'Information' -->
-                      {% set file_detail_url =  file_details_url + '?filename=' + file.filename %}
+                      {%- if file.filename -%}
+                        {% set file_detail_url =  file_details_url + '?filename=' + file.filename %}
+                      {%- else -%}
+                        {% set file_detail_url =  file_details_url + '?file_order=' + (loop.index - 1)|string %}
+                      {%- endif -%}
                       {%- if community -%}
                         {% set file_detail_url = file_detail_url + "&community=" + community.id %}
                       {%- endif -%}

modules/weko-records-ui/weko_records_ui/templates/weko_records_ui/file_details_contents.html

@@ -49,8 +49,11 @@ <h2 class="col-sm-12 col-md-12 col-left" style="margin-top: unset;">{{ filename.
             {%- set ignore_meta = ('_buckets', '_deposit', '_oai', 'path', 'filemeta', 'item_title', 'item_type_id')
             -%}
             {%- if files -%}
-              <!-- Preview Disable Carousel-->
-              {{ preview_carousel(record=record, files=files, pid=pid, filename=filename, nav=False) }}
+              {%- if request.args.get("file_order", '-1') == '-1' -%}
+                <!-- Preview Disable Carousel-->
+                {{ preview_carousel(record=record, files=files, pid=pid, filename=filename, nav=False) }}
+                <hr />
+              {%- endif %}
               <hr />
               <!-- Details -->
               <div class="table-responsive">

modules/weko-records-ui/weko_records_ui/utils.py

@@ -591,11 +591,9 @@ def set_message_for_file(p_file):
                 if check_file_download_permission(record, f, True)\
                         or is_open_restricted(f):
                     # Set default version_id.
-                    if f.get("version_id") is None:
-                        f["version_id"] = ''
+                    f["version_id"] = f.get('version_id', '')
                     # Set is_thumbnail flag.
-                    if f.get("is_thumbnail") is None:
-                        f["is_thumbnail"] = False
+                    f["is_thumbnail"] = f.get('is_thumbnail', False)
                     # Check Opendate is future date.
                     set_message_for_file(f)
                     # Check show preview area.