https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Katrin Fischer changed:
What|Removed |Added
Text to go in the||When choosing the CSV
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Fridolin Somers changed:
What|Removed |Added
Status|Pushed to main |Needs documenting
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #34 from Katrin Fischer ---
Pushed for 24.05!
Well done everyone, thank you!
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Katrin Fischer changed:
What|Removed |Added
Status|Passed QA |Pushed to main
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Katrin Fischer changed:
What|Removed |Added
Keywords||release-notes-needed
--
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Katrin Fischer changed:
What|Removed |Added
Summary|Optionally remove HTML from |Remove HTML from downloaded
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #19 from George Williams (NEKLS) ---
I want to reiterate that whether or not the HTML is stripped from a report when
it's downloaded should be optional. There are times that I actually want the
raw HTML output in a
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #18 from Andrew Fuerste-Henry ---
It would also be great to have HTML-stipping available in reports run and
emailed via the commandline. Martin, is that reasonably easy to add here, or
would you rather I file a new bug?
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #17 from Andrew Fuerste-Henry ---
Clearly I've doing this too early in the morning. I applied your patches to my
testing docker and then ran my reports on the wrong Koha.
When I follow my test plan correctly, this works!
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #16 from Andrew Fuerste-Henry ---
Martin, can you please provide a test plan? What I've tried doesn't seem to
work, but perhaps I'm misunderstanding your intent.
Here's what I did:
- apply patch
- cpan HTML:Restrict
-
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #15 from Andrew Fuerste-Henry ---
(In reply to Martin Renvoize from comment #14)
> Sandboxes don't pull new dependencies from the cpanfile I'm afraid, so this
> one can't easily be tested there :-(.
Ahha! I'll try to get
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Martin Renvoize changed:
What|Removed |Added
Status|Failed QA |Needs Signoff
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Martin Renvoize changed:
What|Removed |Added
Keywords||no-sandbox
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Andrew Fuerste-Henry changed:
What|Removed |Added
Status|Needs Signoff |Failed QA
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #12 from Martin Renvoize ---
Whilst I agree, the core reason for this has an alternative now, I could see
other use cases for HTML in report output that may not be catered for in other
ways yet, so I think this is still a
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Martin Renvoize changed:
What|Removed |Added
Assignee|gmcha...@gmail.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #11 from Martin Renvoize ---
Attached patches from Mirjam's branch.. I wasn't aware of this bug prior to
now, otherwise I would have submitted this up earlier.
--
You are receiving this mail because:
You are the QA
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Martin Renvoize changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #10 from Martin Renvoize ---
Created attachment 161710
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161710=edit
Bug 5920: Add HTML::Restrict to cpanfile
Signed-off-by: Martin Renvoize
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #9 from Martin Renvoize ---
Created attachment 161709
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161709=edit
Bug 5920: Strip HTML from report exports
This patch uses HTML::Restrict to strip out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #8 from Mirjam Vantieghem ---
Addition: The patch from bug #5697 has solved the issue for us, as we were not
using HTML for another purpose in these reports.
--
You are receiving this mail because:
You are the QA
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Mirjam Vantieghem changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Katie Bliss changed:
What|Removed |Added
CC||kebl...@dmpl.org
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Rebecca Coert changed:
What|Removed |Added
CC||rco...@arlingtonva.us
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Andrew Fuerste-Henry changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
George Williams (NEKLS) changed:
What|Removed |Added
CC||geo...@nekls.org
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Séverine Queune changed:
What|Removed |Added
CC||severine.que...@bulac.fr
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
Katrin Fischer changed:
What|Removed |Added
Summary|remove html from downloaded |Remove HTML from downloaded
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #2 from Nicole C. Engard neng...@gmail.com ---
This is how we do it now - but that means 2 reports - one for viewing and
clicking and one for exporting. The library wanted way to use the same report
for both (clicking and
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5920
--- Comment #1 from Owen Leonard oleon...@myacpl.org ---
You would probably have to fully re-build the SQL for output as CSV. If you
simply stripped out the concatenated text you would potentially be outputting
two or more fields
30 matches
Mail list logo
