[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-12-16 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Katrin Fischer  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=35585

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-10-17 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #29 from Matt Blenkinsop  ---
Nice work everyone!

Pushed to oldstable for 22.11.x

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-10-17 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Matt Blenkinsop  changed:

   What|Removed |Added

 Status|Pushed to stable|Pushed to oldstable
 Version(s)|23.11.00,23.05.05   |23.11.00,23.05.05,22.11.11
released in||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-10-05 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Fridolin Somers  changed:

   What|Removed |Added

 CC||fridolin.som...@biblibre.co
   ||m
   Patch complexity|--- |Small patch
 Status|Pushed to master|Pushed to stable
 Version(s)|23.11.00|23.11.00,23.05.05
released in||

--- Comment #28 from Fridolin Somers  ---
Pushed to 23.05.x for 23.05.05

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-10-04 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #27 from Tomás Cohen Arazi  ---
Pushed to master for 23.11.

Nice work everyone, thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-10-04 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Version(s)||23.11.00
released in||
 Status|Passed QA   |Pushed to master

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-26 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Jonathan Druart  changed:

   What|Removed |Added

 Attachment #156195|0   |1
is obsolete||

--- Comment #26 from Jonathan Druart  ---
Created attachment 156212
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156212=edit
Bug 34645: (follow-up) save sort1 and sort2 in order

Signed-off-by: David Nind 

Signed-off-by: Jonathan Druart 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-26 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Jonathan Druart  changed:

   What|Removed |Added

 Attachment #156194|0   |1
is obsolete||

--- Comment #25 from Jonathan Druart  ---
Created attachment 156211
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156211=edit
Bug 34645: always use discount, sort1, and sort2 from MarcFieldsToOrder

These values are stored at the order level, not per item. This patch simply
sets the order values from the values retrieved from MarcFieldsToOrder even inf
MarcItemFields to order is being used for other fields

To test:
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l

Stage the attached bib-303.marcxml file
Add to basket from the staged file
Note that discount and sort1 and sort2 are not populated per the Marc
Apply patch, restart all
Stage and add to basket again
Confirm discount and sort1 and sort2 are populated correctly

Signed-off-by: David Nind 

Signed-off-by: Jonathan Druart 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-26 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Jonathan Druart  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

David Nind  changed:

   What|Removed |Added

 Attachment #156156|0   |1
is obsolete||

--- Comment #24 from David Nind  ---
Created attachment 156195
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156195=edit
Bug 34645: (follow-up) save sort1 and sort2 in order

Signed-off-by: David Nind 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

David Nind  changed:

   What|Removed |Added

 Attachment #155852|0   |1
is obsolete||

--- Comment #23 from David Nind  ---
Created attachment 156194
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156194=edit
Bug 34645: always use discount, sort1, and sort2 from MarcFieldsToOrder

These values are stored at the order level, not per item. This patch simply
sets the order values from the values retrieved from MarcFieldsToOrder even inf
MarcItemFields to order is being used for other fields

To test:
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l

Stage the attached bib-303.marcxml file
Add to basket from the staged file
Note that discount and sort1 and sort2 are not populated per the Marc
Apply patch, restart all
Stage and add to basket again
Confirm discount and sort1 and sort2 are populated correctly

Signed-off-by: David Nind 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

David Nind  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #22 from David Nind  ---
Thanks Nick!

I was able to successfully run through the test plan with and without deleting
the existing record. These are the settings I used when staging the file:
- Record matching rule: KohaBiblio (999$c)
- Action if matching record found: Ignore incoming record (its items may still
be processed)
- Check for embedded item record data? Yes + How to process items: Always add
items

I can now see the statistics columns in the basket (used Columns to toggle on
the display of the Statistics 1 and Statistics 2 columns).

David


Latest testing notes (using KTD):

1. Works successfully without needing to delete the existing record 303 (Pawn
structure chess). Also works successfully if the existing record is deleted.

2. With existing record and items, use these settings when staging the file:
   - Record matching rule: KohaBiblio (999$c)
   - Action if matching record found: Ignore incoming record (its items may
still be processed)
   - Check for embedded item record data? Yes + How to process items: Always
add items

3. Steps:
   3.1 Remember to set up the MarcFieldsToOrder and MarcItemFieldsToOrder
system preferences as per the test plan!
   3.2 Stage the file provided - use these settings (as the record and four
items already exist):
  - Record matching rule: KohaBiblio (999$c)
  - Action if matching record found: Ignore incoming record (its items may
still be processed)
  - Check for embedded item record data? Yes + How to process items: Always
add items   
   3.3 Create a new basket for the vendor, for 'Create items when' select
"placing an order" .
   3.4 Add to the new basket using the 'From a staged file' option.
   3.5 Tick item 'Pawn structure chess'
   - under the 'match heading' it shows quantity of 8 and correct discount
+ Sort1 and Sort2 in Statistics 1 and 2
  - on the right-hand side, it lists 8 item records with the correct
budgets, prices, etc., from the staged file.
   3.6 Change to the 'Item information' tab, select a Koha item type (work
around for bug 34827).
   3.7 Change back to the 'Select to import' tab.
   3.8 Click 'Save' at the bottom of the page.
   3.9 Result ==> Items are added to the basket:
   - discount correctly shown on the order lines
   - use the Columns setting to display the Statistics 1 and Statistics 2
columns.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #21 from Nick Clemens  ---
Created attachment 156156
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156156=edit
Bug 34645: (follow-up) save sort1 and sort2 in order

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #20 from Nick Clemens  ---
(In reply to David Nind from comment #19)
> 1. I can successfully test by deleting the existing items and record. Should
> this work where an existing record with items exists? I tried leaving the
> existing record and items in place, matched on 999$c. It matched the record,
> but didn't show the 8 new items. I'm assuming how this works is determined
> by what is in the file, and that there is no change in behavour here.

This works for me - the matching rule should be set when staging, and then
using the same during adding to basket - I get 12 items on the record. I would
make sure the basket/vendor is set to create items on ordering
> 
> 2. I found the statistics codes under the new record in 949$n and 949$q - is
> that the correct place to look for these? I couldn't see them anywhere else.

Attaching a followup - while these are set as variables when using
MarcItemFieldsToOrder it looks like they were never saved into the order

> 
> 3. The new record created uses the default framework. I'm assuming that this
> is what is expected.

Yes, at the moment there is no choice, that's a future/separate enhancement

> Happy to sign off otherwise.
> And apologies for the questions, I know the basics for acquisitions, but not
> much more

Questions are good! And thank you for testing (many times)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #19 from David Nind  ---
Thanks Nick!

I have had another go at testing, and it works now (yeah!). 

I have a couple of questions though:

1. I can successfully test by deleting the existing items and record. Should
this work where an existing record with items exists? I tried leaving the
existing record and items in place, matched on 999$c. It matched the record,
but didn't show the 8 new items. I'm assuming how this works is determined by
what is in the file, and that there is no change in behavour here.

2. I found the statistics codes under the new record in 949$n and 949$q - is
that the correct place to look for these? I couldn't see them anywhere else.

3. The new record created uses the default framework. I'm assuming that this is
what is expected.

Happy to sign off otherwise.

And apologies for the questions, I know the basics for acquisitions, but not
much more

David


Testing notes (so far):

1. Deleted record 303 (Pawn structure chess), including items.

2. Information is now included when adding to the basket from the staged record
(discount and sort1 and sort2) + the funds etc., for the item records.

3. 8 item records are shown, with fund details, prices, etc., from the imported
file.

4. Steps:
   4.1 Deleted items and then the record for Pawn structure chess.
   4.2 Staged the file provided - didn't change any of the settings.
   4.3 Added to basket already setup in KTD - from a staged file option.
   4.4 Ticked item 'Pawn structure chess'
   - under the 'match heading' it shows quantity of 8 and correct discount
+ Sort1 and Sort in Statistics 1 and 2
  - on the right-hand side, it lists Item record 1 to 8
   4.5 Changed to the 'Item information' tab, to select the Koha item type
(work around for bug 34827).
   4.6 Changed back to the 'Select to import' tab.
   4.7 Clicked 'Save' at the bottom of the page.
   4.8 result: Items are added to the basket:
   - discount correctly shown on the order lines
   - statistics shown in the record (open title in a new tab, edit record,
statistics codes shown under 949$n and 949$q)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #18 from Nick Clemens  ---
(In reply to David Nind from comment #17)
> Now I'm getting "Some required item subfields are not set" when I go to save.

That is an preexisting bug, you can workaround by setting the item type on the
second tab

I filed the issue here:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34827

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #17 from David Nind  ---
Now I'm getting "Some required item subfields are not set" when I go to save.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #16 from David Nind  ---
(In reply to Nick Clemens from comment #15)
> 
> We expect 8 items - each order line has a quantity of 2
> 
> If you complete adding them you get three orders, 2 for 2 and one for 4 -
> because we group the items by budget I believe

Thanks Nick! That explains it(and makes sense now).

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #15 from Nick Clemens  ---
(In reply to Katrin Fischer from comment #14)
> (In reply to David Nind from comment #13)
> 
> > I'm sure it is something I'm doing wrong!
> 
> I am not so sure about this!

We expect 8 items - each order line has a quantity of 2

If you complete adding them you get three orders, 2 for 2 and one for 4 -
because we group the items by budget I believe

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #14 from Katrin Fischer  ---
(In reply to David Nind from comment #13)

> I'm sure it is something I'm doing wrong!

I am not so sure about this!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #13 from David Nind  ---
I had a go at testing (using KTD):

1. Deleted record 303, including items.

2. Information is now included when adding to the basket from the staged record
(discount and sort1 and sort2) + the funds etc for the item records.

3. However, 8 item records are shown, instead of the 4 that are in the file.
Item record 1 and 2 are duplicated (from libraries and funds), and so on.

3. Steps:
   3.1 Deleted items and then the record for Pawn structure chess
   3.2 Staged the file provided - didn't change any of the settings
   3.3 Added to basket already setup in KTD - from a staged file
   3.4 Ticked item 'Pawn structure chess'
   - under match heading shows quantity of 8 and correct discount + Sort1
and Sort in Statistics 1 and 2
  - on the right-hand side, it lists Item record 1 to 8

I'm sure it is something I'm doing wrong!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #12 from Katrin Fischer  ---
Hi Nick, so it wasn't just me? Did you figure out what went wrong?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Nick Clemens  changed:

   What|Removed |Added

 Attachment #155564|0   |1
is obsolete||

--- Comment #11 from Nick Clemens  ---
Created attachment 155852
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155852=edit
Bug 34645: always use discount, sort1, and sort2 from MarcFieldsToOrder

These values are stored at the order level, not per item. This patch simply
sets the order values from the values retrieved from MarcFieldsToOrder even inf
MarcItemFields to order is being used for other fields

To test:
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l

Stage the attached bib-303.marcxml file
Add to basket from the staged file
Note that discount and sort1 and sort2 are not populated per the Marc
Apply patch, restart all
Stage and add to basket again
Confirm discount and sort1 and sort2 are populated correctly

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Nick Clemens  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-17 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Katrin Fischer  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #10 from Katrin Fischer  ---
I've made sure the patch was applied, restart_all, but the statistic fields and
discount don't get filled for me. I tried several times, with 'Order from new
file' and 'Order from staged file'. 

I can tell the 9xx fields are still present in the staged file.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-15 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Nick Clemens  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

--- Comment #9 from Nick Clemens  ---
(In reply to Esther Melander from comment #7)
> I ran through the test plan and it did appear to work. However the
> bibliographic record in the example import file was already in the system. I
> had to delete the existing bib and items records from the system before
> adding the order from the staged file to the basket so that everything came
> through as new records.
> 
> If the order is placed from the staged file with the bibliographic record
> already in the system previously, you are redirected to treating each item
> record individually. In this case, the discounted price does not seem to
> come through. I'm not 100% sure on the correct workflow for this scenario.
> Only mentioning because some orders may actually be reorders for existing
> records.

The order process has two places you can set matching - during staging, or when
adding orders to the cart. Generally I would say matching should be set during
staging and not during adding to basket, or you hit this problem.

This is an existing workflow issue, outside of this bug

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-15 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Marcel de Rooy  changed:

   What|Removed |Added

 CC||m.de.r...@rijksmuseum.nl
 Status|Signed Off  |Failed QA

--- Comment #8 from Marcel de Rooy  ---
(In reply to Esther Melander from comment #7)
> I ran through the test plan and it did appear to work. However the
> bibliographic record in the example import file was already in the system. I
> had to delete the existing bib and items records from the system before
> adding the order from the staged file to the basket so that everything came
> through as new records.
> 
> If the order is placed from the staged file with the bibliographic record
> already in the system previously, you are redirected to treating each item
> record individually. In this case, the discounted price does not seem to
> come through. I'm not 100% sure on the correct workflow for this scenario.
> Only mentioning because some orders may actually be reorders for existing
> records.

Does this still need attention, Kyle?

Changing status for feedback.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Esther Melander  changed:

   What|Removed |Added

 CC||esther.melander@bywatersolu
   ||tions.com
 Status|Needs Signoff   |Signed Off

--- Comment #7 from Esther Melander  ---
I ran through the test plan and it did appear to work. However the
bibliographic record in the example import file was already in the system. I
had to delete the existing bib and items records from the system before adding
the order from the staged file to the basket so that everything came through as
new records.

If the order is placed from the staged file with the bibliographic record
already in the system previously, you are redirected to treating each item
record individually. In this case, the discounted price does not seem to come
through. I'm not 100% sure on the correct workflow for this scenario. Only
mentioning because some orders may actually be reorders for existing records.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #154912|0   |1
is obsolete||

--- Comment #6 from ByWater Sandboxes  ---
Created attachment 155564
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155564=edit
Bug 34645: Add support for discount, sort1 and sort2 to MarcItemFieldsToOrder

MarcItemFieldsToOrder supersedes MarcFieldsToOrder, so the fields discount,
sort1, and sort2 won't be populated if using the latter - We should add these
fields in to allow the orders to be created correctly

Test Plan:
1) Set up MarcItemFields to order
2) Map the new keys discount, sor1 and sort2 to fields in the order
   record
3) Import an order record into a basket
4) Note those fields were imported!

Signed-off-by: Esther 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #5 from Nick Clemens  ---
Created attachment 155338
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155338=edit
Sample file with order lines

This is a pretty standard plan and file for testing, it could (should) be added
to the wiki for future reference

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #4 from Nick Clemens  ---
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l


Stage the attached bib-303.marcxml file
Add to basket from the staged file
Note that discount and sort1 and sort2 are not populated per the Marc
Apply patch, restart all
Stage and add to basket again
Confirm discount and sort1 and sort2 are populated correctly

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

David Nind  changed:

   What|Removed |Added

 CC||da...@davidnind.com

--- Comment #3 from David Nind  ---
(In reply to David Cook from comment #2)
> (In reply to Kyle M Hall from comment #1)
> 
> I don't understand any of these steps .
> 
> Could we get a more detailed test plan?

This article may help - Automatically Generate Item Information for Order Files
https://bywatersolutions.com/education/automatically-generate-item-information-for-order-files

If it was possible to provide a sample for importing, I'd be happy to test.

Otherwise, I think it requires setting up some info on a vendor website (Baker
& Taylor, Brodart, Ingram, etc) to identify the MARC item fields with the order
information.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-09-05 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

David Cook  changed:

   What|Removed |Added

 CC||dc...@prosentient.com.au

--- Comment #2 from David Cook  ---
(In reply to Kyle M Hall from comment #1)
> Test Plan:
> 1) Set up MarcItemFields to order
> 2) Map the new keys discount, sor1 and sort2 to fields in the order
>record
> 3) Import an order record into a basket
> 4) Note those fields were imported!

I don't understand any of these steps .

Could we get a more detailed test plan?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-08-31 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Nick Clemens  changed:

   What|Removed |Added

   Severity|normal  |major

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-08-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Kyle M Hall  changed:

   What|Removed |Added

 Blocks||34644
 Depends on|34644   |


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34644
[Bug 34644] Add clarifying text to sysprefs to indicate that MarcFieldsToOrder
is a fallback to MarcItemFieldsToOrder
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-08-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Kyle M Hall  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com
   |ity.org |
 CC||k...@bywatersolutions.com

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-08-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

--- Comment #1 from Kyle M Hall  ---
Created attachment 154912
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154912=edit
Bug 34645: Add support for discount, sort1 and sort2 to MarcItemFieldsToOrder

MarcItemFieldsToOrder supersedes MarcFieldsToOrder, so the fields discount,
sort1, and sort2 won't be populated if using the latter - We should add these
fields in to allow the orders to be created correctly

Test Plan:
1) Set up MarcItemFields to order
2) Map the new keys discount, sor1 and sort2 to fields in the order
   record
3) Import an order record into a basket
4) Note those fields were imported!

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-08-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Kyle M Hall  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 34645] Add missing fields to MarcItemFieldsToOrder system preference

2023-08-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34645

Nick Clemens  changed:

   What|Removed |Added

 Depends on||34644


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34644
[Bug 34644] Add text to sysprefs indicate that MarcFieldsToOrder is not used if
MarcItemFieldsToOrder is populated
-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/