[Koha-bugs] [Bug 9659] Undefined authorised value category yields empty dropdown menu on SQL reports

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9659

Tomás Cohen Arazi <tomascohen at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #15560|0                           |1
        is obsolete|                            |

--- Comment #3 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
Created attachment 15596
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15596&action=edit
Bug 9659 - Better handling of non-existent authorised value categories used in
SQL reports

A user might create a SQL report that relies on non-existent authorised value
categories.
Because of a typo, or just because they copy&pasted the report from the Wiki.

Use cases are:

- The user creates a report from SQL
  a) Uses bad authorised values
  b) Clicks 'Save Report'
  c) Koha lists the problematic authorised values
  d) The user decides to
  e-1) Save it anyway, it gets saved
  e-2) Edit the report, it gets back to where it chose 'Save Report'

- The user edits an already saved report (Update SQL)
  a) Uses bad authorised values
  b) Clicks 'Update SQL'
  c) Koha lists the problematic authorised values
  d) The user decides to
  e-1) Save it anyway, it gets saved
  e-2) Edit the report, it gets back to where it chose 'Update SQL'

- The user tries to run a saved report that contains bad authorised values,
Koha advertises the problem and provides the user with a button 'Edit SQL' to
fix things.

Notes:
- I added several comments on the code.
- Fixed an annoying warning of uninitialised variable also (refactored some
tiny bits to do it).
- Added the following methods
   -  C4::Reports::Guided::GetReservedAuthorisedValues
   -  C4::Reports::Guided::GetParametersFromSQL
   -  C4::Reports::Guided::IsAuthorisedValueValid
   -  C4::Reports::Guided::ValidateSQLParameters
   -  C4::Koha::IsAuthorisedValueCategory
- Those methods could have been used to refactor this guided reports code as
its *a bit messy*. I chose to do it in a new bug of course :-D.
- Thanks to Owen and Jared for their patience heh.

Sponsored-by: Universidad Nacional de Córdoba

-- 
You are receiving this mail because:
You are watching all bug changes.