Functionality of submit button issue

visolveemr · October 11, 2017, 4:12am

@brady.miller
Viewing reports page of Unique Seen Patients provide a perplexed behaviour. Steps followed are given below.

Click Reports->Insurance->unique Sp.
On clicking submit, displays the list.
On clicking labels, prompts to download the txt file. But again on changing date and clicking on submit,it prompts to save file instead of listing the content.

Thanks,
ViSolve

brady.miller · October 11, 2017, 9:15am

Hi @visolveemr ,

As I recall, I noted that behavior on some other reports. I think the fix would be here:

github.com

openemr/openemr/blob/master/interface/reports/unique_seen_patients_report.php#L168


                               <?php echo xlt('Print'); ?>
                       </a>
                       <a href='#' class='btn btn-default btn-transmit' onclick='$("#form_labels").attr("value","true"); $("#theform").submit();'>
                           <?php echo xlt('Labels'); ?>
                       </a>
                   <?php } ?>
        </div>
              </div>
          </td>
      </tr>
  </table>
</td>
</tr>
</table>
</div> <!-- end of parameters -->


<div id="report_results">
<table>


<thead>
<th> <?php echo xlt('Last Visit'); ?> </th>

With something like:

<a href='#' class='btn btn-default btn-save' onclick='$("#form_refresh").attr("value","true"); $("#form_labels").attr("value","false"); $("#theform").submit();'>

(basically setting form_labels to false or else it keeps getting carried over in the form variable)

-brady

visolveemr · October 12, 2017, 9:05am

@brady.miller
Thanks for you inputs. We did check with the code you shared but that did not work as expected.
We just tweeked the code a bit and it works good.
Kindly check.
Here is the updated code for your reference.
<a href='#' class='btn btn-default btn-save' onclick='$("#form_refresh").attr("value","true");$("#form_labels").val(""); $("#theform").submit();'>

Thanks,
ViSolve

brady.miller · October 15, 2017, 8:32am

Hi @visolveemr ,
Just brought this fix into the codebase.
thanks!
-brady

visolveemr · October 16, 2017, 4:14am

@brady.miller

Thanks brady

Thanks,
ViSolve