jaredcowing / wmsNewBooks

Small program to view list of new books by fund & date acquired using WorldShare Acquisitions API.
MIT License
2 stars 1 forks source link

Bookview: replace echo calls with append_output() #9

Closed gjerdery closed 4 years ago

gjerdery commented 4 years ago

I was noticing in my Bookview display that the page content was appearing ahead of the header and footer content if viewing the page source HTML. I found that the load->view() method used for headers and footers appends file content to the output stream, which is then evaluated at the end of the function.

The included patch appends the html strings to the output stream, so that the page gets constructed in the expected order.

jaredcowing commented 4 years ago

Good catch, thank you!