[Koha-bugs] [Bug 9281] Impossible to create an order from a staged file

2013-02-01 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Chris Cormack ch...@bigballofwax.co.nz changed:

   What|Removed |Added

 Status|Pushed to Master|Pushed to Stable
 CC||ch...@bigballofwax.co.nz

--- Comment #15 from Chris Cormack ch...@bigballofwax.co.nz ---
Pushed to 3.10.x and 3.8.x will be in 3.10.3 and 3.8.10

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-22 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #11 from Jonathan Druart jonathan.dru...@biblibre.com ---
Created attachment 14738
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14738action=edit
test file

Hi Katrin,

Could you test this perl script on master please?

  perl test_bug9281.pl

gives me

  DBD::mysql::st execute failed: called with 2 bind variables when 1 are needed
at /home/koha/src/C4/Members.pm line 560.
  Can't use an undefined value as an ARRAY reference at /usr/lib/perl5/DBI.pm
line 2054.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-22 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #12 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Hi Jonathan,

thank you for your patience. The file you provided gives me the error when
running it and I finally found the error in my log files on ordering from a
staged record:

[Wed Jan 23 08:25:51 2013] [error] [client 127.0.0.1] [Wed Jan 23 08:25:51
2013] addorderiso2709.pl: DBD::mysql::st execute failed: called with 2 bind
variables when 1 are needed at /home/katrin/kohaclone/C4/Members.pm line 559.,
referer:
http://localhost:8080/cgi-bin/koha/acqui/addorderiso2709.pl?import_batch_id=15basketno=10booksellerid=2

I still wonder, what kind of problem this will cause, as it seems that the
acquisition module works correctly, with and without the patch. 

But there is nothing to say against the patch - passing QA :)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-22 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Katrin Fischer katrin.fisc...@bsz-bw.de 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
http://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 9281] Impossible to create an order from a staged file

2013-01-22 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Katrin Fischer katrin.fisc...@bsz-bw.de changed:

   What|Removed |Added

  Attachment #14139|0   |1
is obsolete||

--- Comment #13 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Created attachment 14767
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14767action=edit
[SIGNED-OFF] Bug 9281: FIX add an order from a staged file

In GetMember the hash parameter contained
'C4::Members' = 'borrowernumber'
XXX = undef

Test plan:
In the acquition module, try to add an order to a basket from a staged file.
Without this patch, you get an error:
  DBD::mysql::st execute failed: called with 2 bind variables when 1 are needed
at /home/koha/src/C4/Members.pm line 559

Signed-off-by: Kyle M Hall k...@bywatersolutions.com
Signed-off-by: Katrin Fischer katrin.fischer...@web.de
All tests pass, error only appearing in the log file.

Test plan:
1) Create a basket
2) Add one or multiple order lines using a staged file and the 'batch' feature.
   This is done by using the Save button at the bottom of the page instead
   of the Add order link next to a single entry.
3) Check your koha-error_log for the error above.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-22 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Katrin Fischer katrin.fisc...@bsz-bw.de changed:

   What|Removed |Added

   Patch complexity|--- |Small patch
 QA Contact||katrin.fisc...@bsz-bw.de

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-20 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #9 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Hi Jonathan, I am not sure I did it right, as my output looks quite different
to yours. I added it at the beginning of the GetMembers routing, but the line
number was totally different to the one you noted in your comment. Could you
check?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-17 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #8 from Jonathan Druart jonathan.dru...@biblibre.com ---
(In reply to comment #7)

 You mean the save button at the bottom of the page below the accounting
 details box?
Yes

 I successfully added all records from a staged file as orders
 that way - on error. Is there another condition that has to be met to
 trigger the error?

hum... strange.
Could you dump the hash in GetMember please?
(in C4/Members.pm l.530: warn Data::Dumper::Dumper \%information;)
I get:
  $VAR1 = {
'C4::Members' = 'borrowernumber',
'55' = undef, # 55 is $borrowernumber
  }
According to the code, you should enter in this routine for each biblio
contained in your staged file.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-16 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #6 from Jonathan Druart jonathan.dru...@biblibre.com ---
(In reply to comment #5)
 What am I missing?

Hi Katrin,
Yep I forgot 1 thing, on the Add orders from page, click on the Save button
instead of the Add order link.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-16 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #7 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Hi Jonathan,

I just tried it again and I think I already did it yesterday:

I also tried adding the rest of the staged file as batch and the orders were 
created perfectly.

You mean the save button at the bottom of the page below the accounting details
box? I successfully added all records from a staged file as orders that way -
on error. Is there another condition that has to be met to trigger the error?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2013-01-15 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Katrin Fischer katrin.fisc...@bsz-bw.de changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

--- Comment #5 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Hi Jonathan,

I am sorry, but I can't reproduce the problem described. 

I added 2 order lines from a file I had staged some time ago and no error
appeared.

I also tried adding the rest of the staged file as batch and the orders were
created perfectly.

The logs are clean too.

AcqCreateItem is set to 'on receive'.

What am I missing?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2012-12-14 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

  Attachment #14070|0   |1
is obsolete||

--- Comment #2 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 14139
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14139action=edit
Bug 9281: FIX add an order from a staged file

In GetMember the hash parameter contained
'C4::Members' = 'borrowernumber'
XXX = undef

Test plan:
In the acquition module, try to add an order to a basket from a staged file.
Without this patch, you get an error:
  DBD::mysql::st execute failed: called with 2 bind variables when 1 are needed
at /home/koha/src/C4/Members.pm line 559

Signed-off-by: Kyle M Hall k...@bywatersolutions.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2012-12-14 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off
 CC||k...@bywatersolutions.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2012-12-14 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #3 from Owen Leonard oleon...@myacpl.org ---
Created attachment 14143
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14143action=edit
Bug 9284 - JavaScript should follow rules recommended by JSHint

Coding style corrections to members.js:

- Mixed tab/space indentation converted to spaces.
- Converted != and == to !== and === for comparisons with ''.
- Adding missing semicolons.

To test, create or edit a patron and confirm that validation hasn't been
broken by the changes to members.js: Invalid date, empty required
fields, guarantor popup, restricted-until field display, etc.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2012-12-14 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Owen Leonard oleon...@myacpl.org changed:

   What|Removed |Added

  Attachment #14143|0   |1
is obsolete||

--- Comment #4 from Owen Leonard oleon...@myacpl.org ---
Comment on attachment 14143
  -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14143
Bug 9284 - JavaScript should follow rules recommended by JSHint

Sorry, wrong bug number!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 9281] Impossible to create an order from a staged file

2012-12-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Jonathan Druart jonathan.dru...@biblibre.com 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
http://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 9281] Impossible to create an order from a staged file

2012-12-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

--- Comment #1 from Jonathan Druart jonathan.dru...@biblibre.com ---
Created attachment 14070
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14070action=edit
Bug 9281: FIX add an order from a staged file

In GetMember the hash parameter contained
'C4::Members' = 'borrowernumber'
XXX = undef

Test plan:
In the acquition module, try to add an order to a basket from a staged file.
Without this patch, you get an error:
  DBD::mysql::st execute failed: called with 2 bind variables when 1 are needed
at /home/koha/src/C4/Members.pm line 559

-- 
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
http://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 9281] Impossible to create an order from a staged file

2012-12-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9281

Jonathan Druart jonathan.dru...@biblibre.com changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |jonathan.dru...@biblibre.co
   |ity.org |m

-- 
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
http://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/