[Koha-bugs] [Bug 10612] Add ability to delete patrons with batch patron modification tool

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

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

   What|Removed |Added

 Status|Signed Off  |In Discussion

--- Comment #16 from Jonathan Druart jonathan.dru...@biblibre.com ---
ping Kyle, park to in discussion.

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

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

--- Comment #18 from Jonathan Druart jonathan.dru...@biblibre.com ---
(In reply to Kyle M Hall from comment #17)
 The code is an extension of the addition of lists to the patron modification
 tool. I understand what you mean. However I'd suggest that over time we,
 instead of having two separate tools that function in different ways, unify
 both the existing patron modification and deletion tools in a manner
 equivalent to the batch item and modification deletion which is just the
 same script with a flag passed in for deletion.

I don't think the batch item mod/del is a good example :) The code is not clean
and not easy to read/understand.
I think it is a good thing to keep 2 pages and 2 scripts: 1 for the deletion
and 1 for the modification.
The borrower modification script has 400 lines and will be difficult to
maintain if it grows too much.
Moreover the Delete patrons checkbox is not ergonomic, I think: the user can
edit patron fields and at the same time, delete patrons.

Maybe should we have another point of view.

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

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

--- Comment #19 from Nora Blake nbl...@masslibsystem.org ---
Hello,

I am the force behind this whole development and I can tell you what our
situation is and what we are trying to accomplish.  We have many schools in our
consortium.  They want to be able to remove classes of students each year after
they graduate.  Our schools do not treat their patron accounts in the same way
that they treat books, so they do not have the student barcodes on hand or
available in the same way that they can grab a stack of books and scan them all
in to the batch item deletion tool.

They need to be able to search for a group of patrons by a common piece of
information stored in a single field.  In our case, we store year of graduation
information in either the sort 1 or sort 2 field of the patron record. 
They need to be able to search for that common piece of information easily,
obtain a grouping of patrons from the search and then decide whether they want
to delete the whole class of kids from the system (probably the most common
event we will have) OR change their patron type from student to alumni or
something else.

They don't use patron card numbers to find patron accounts.  Often they
checkout based on a name search or they may scan the kid's school ID, but they
won't have those ID cards on hand when it comes time to delete them from the
system.  So they need to be able to perform a search for the class of 2014 or
2014 or grade 12, depending on what they have entered in the child's record
and get a group of patrons together in one place so they can then delete them
(most common need) or modify their patron type or modify their year of
graduation if several are being held back or do something else that I can't
think of.

I hope this helps clarify why Kyle is looking to do what he is proposing.

Thanks,

Nora Blake

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-11-12 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

--- Comment #15 from Jonathan Druart jonathan.dru...@biblibre.com ---
Kyle,
Why don't you use the Batch patron deletion? Maybe we could have a new entry
Delete patrons in the list select.
Don't you think it is not consistent to allow to delete patron in 2 different
places?

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-11-08 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Status|Patch doesn't apply |Signed Off

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-11-08 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

  Attachment #21081|0   |1
is obsolete||

--- Comment #13 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 22837
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22837action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool

The batch patron deletion/anonymization does not allow for batch
deletion of arbitrary lists of patrons. The batch patron modification
tool allows for modification of arbitrary lists of patrons, but not
deletion. If would be highly beneficial to add patron deletion to the
batch patron modification tool.

Test Plan:
1) Apply all dependencies for this patch
2) Apply this patch
3) Create a list of patrons with the new Patron Lists feature
   a) Include at least one patron owing fines
   b) Include at least one patron with items currently checked out
   c) Include at least one patron not falling into a) or b)
4) Browse to the batch patron modifications tool
5) Select your list from the pulldown, and submit
6) Check the Delete patrons checkbox, the click the submit button
7) You should a list of errors for the patrons with fines or issues
   and a table of patrons that were deleted successfully.
8) Click the link for a deleted patron, you should get a patron not
   found message.

Signed-off-by: Nora Blake nbl...@masslibsystem.org

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-11-08 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

  Attachment #21446|0   |1
is obsolete||

--- Comment #14 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 22838
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22838action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool -
QA Followup - Move member deletion related calls to DelMember

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-28 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Status|Signed Off  |Patch doesn't apply
 CC||katrin.fisc...@bsz-bw.de

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

I am sorry, but this patch no longer applies and I was not sure how to resolve
the conflict, could you please take a look?

[~/kohaclone (73-10612-deletepatronsbatch)] git bz apply 10612
Bug 10612 - Add ability to delete patrons with batch patron modification tool

21081 - Bug 10612 - Add ability to delete patrons with batch patron
modification tool
21446 - Bug 10612 - Add ability to delete patrons with batch patron
modification tool - QA Followup - Move member deletion related calls to
DelMember

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 10612 - Add ability to delete patrons with batch patron
modification tool
/home/katrin/kohaclone/.git/rebase-apply/patch:41: trailing whitespace.
Cannot delete patron 
/home/katrin/kohaclone/.git/rebase-apply/patch:49: trailing whitespace.
Cannot delete patron 
/home/katrin/kohaclone/.git/rebase-apply/patch:101: trailing whitespace.

warning: 3 lines add whitespace errors.
Using index info to reconstruct a base tree...
stdin:41: trailing whitespace.
Cannot delete patron 
stdin:49: trailing whitespace.
Cannot delete patron 
stdin:101: trailing whitespace.

warning: 3 lines applied after fixing whitespace errors.
Falling back to patching base and 3-way merge...
Auto-merging tools/modborrowers.pl
CONFLICT (content): Merge conflict in tools/modborrowers.pl
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
Failed to merge in the changes.
Patch failed at 0001 Bug 10612 - Add ability to delete patrons with batch
patron modification tool
When you have resolved this problem run git bz apply --continue.
If you would prefer to skip this patch, instead run git bz apply --skip.
To restore the original branch and stop patching run git bz apply --abort.
Patch left in
/tmp/Bug-10612---Add-ability-to-delete-patrons-with-bat-_BRqbC.patch

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-25 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Status|Failed QA   |Signed Off

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-25 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

--- Comment #9 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 21444
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21444action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool -
QA Followup - Move member deletion related calls to DelMember

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-25 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

  Attachment #21444|0   |1
is obsolete||

--- Comment #10 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 21445
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21445action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool -
QA Followup - Move member deletion related calls to DelMember

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-25 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

  Attachment #21445|0   |1
is obsolete||

--- Comment #11 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 21446
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21446action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool -
QA Followup - Move member deletion related calls to DelMember

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-19 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Status|Signed Off  |Failed QA
 CC||jonathan.dru...@biblibre.co
   ||m

--- Comment #8 from Jonathan Druart jonathan.dru...@biblibre.com ---
Kyle, could you answer to Galen please (see comment 5).
A call is missing, marked as Failed 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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Melissa Lefebvre meli...@bywatersolutions.com 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
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Melissa Lefebvre meli...@bywatersolutions.com changed:

   What|Removed |Added

  Attachment #19767|0   |1
is obsolete||
 CC||meli...@bywatersolutions.co
   ||m

--- Comment #7 from Melissa Lefebvre meli...@bywatersolutions.com ---
Created attachment 21081
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21081action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

  Attachment #21081|0   |1
   is patch||
  Attachment #21081|application/mbox|text/plain
  mime type||

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-09-09 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Nora Blake nbl...@masslibsystem.org changed:

   What|Removed |Added

 CC||nbl...@masslibsystem.org

--- Comment #6 from Nora Blake nbl...@masslibsystem.org ---
I am extremely happy with this and would like to sign off on it.

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-08-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Nicole C. Engard neng...@gmail.com changed:

   What|Removed |Added

 CC||neng...@gmail.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 10612] Add ability to delete patrons with batch patron modification tool

2013-08-12 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

--- Comment #5 from Galen Charlton gmcha...@gmail.com ---
Comment on attachment 19767
  -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19767
Bug 10612 - Add ability to delete patrons with batch patron modification tool

Review of attachment 19767:
 -- 
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.htmlbug=10612attachment=19767)
-

::: tools/modborrowers.pl
@@ +259,5 @@
 +{ error = fees_owed, borrower = $borrower } );
 +}
 +else {
 +MoveMemberToDeleted($borrowernumber);
 +DelMember($borrowernumber);

This is missing a call to C4::VirtualShelves::HandleDelBorrower.

Also, with the proposed patron purging cronjob (bug 10419) also in play, I am
getting concerned about the number of different places in the code that (a)
check whether a patron record is allowed to be deleted and (b) do the deletion,
particularly since the original patch series for bug 10419 had also missed
HandleDelBorrower.

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-31 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

--- Comment #4 from Kyle M Hall k...@bywatersolutions.com ---
(In reply to Liz Rea from comment #3)
 Hi,
 
 Tried to test this, got the following: 
 
 Falling back to patching base and 3-way merge...
 Auto-merging tools/modborrowers.pl
 CONFLICT (content): Merge conflict in tools/modborrowers.pl
 Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
 Failed to merge in the changes.
 Patch failed at 0001 Bug 10612 - Add ability to delete patrons with batch
 patron modification tool

It applied fine for me. Did you apply all the dependencies? First you need to
apply 8798, then 10565, then finally this patch.

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-28 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Liz Rea l...@catalyst.net.nz changed:

   What|Removed |Added

 Status|Needs Signoff   |Patch doesn't apply
 CC||l...@catalyst.net.nz

--- Comment #3 from Liz Rea l...@catalyst.net.nz ---
Hi,

Tried to test this, got the following: 

Falling back to patching base and 3-way merge...
Auto-merging tools/modborrowers.pl
CONFLICT (content): Merge conflict in tools/modborrowers.pl
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
Failed to merge in the changes.
Patch failed at 0001 Bug 10612 - Add ability to delete patrons with batch
patron modification tool

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-18 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

   Assignee|gmcha...@gmail.com  |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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-18 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Depends on||10565

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-18 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

--- Comment #1 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 19766
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19766action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool

The batch patron deletion/anonymization does not allow for batch
deletion of arbitrary lists of patrons. The batch patron modification
tool allows for modification of arbitrary lists of patrons, but not
deletion. If would be highly beneficial to add patron deletion to the
batch patron modification tool.

Test Plan:
1) Apply all dependencies for this patch
2) Apply this patch
3) Create a list of patrons with the new Patron Lists feature
   a) Include at least one patron owing fines
   b) Include at least one patron with items currently checked out
   c) Include at least one patron not falling into a) or b)
4) Browse to the batch patron modifications tool
5) Select your list from the pulldown, and submit
6) Check the Delete patrons checkbox, the click the submit button
7) You should a list of errors for the patrons with fines or issues
   and a table of patrons that were deleted successfully.
8) Click the link for a deleted patron, you should get a patron not
   found message.

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-18 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
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 10612] Add ability to delete patrons with batch patron modification tool

2013-07-18 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

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

   What|Removed |Added

  Attachment #19766|0   |1
is obsolete||

--- Comment #2 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 19767
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19767action=edit
Bug 10612 - Add ability to delete patrons with batch patron modification tool

The batch patron deletion/anonymization does not allow for batch
deletion of arbitrary lists of patrons. The batch patron modification
tool allows for modification of arbitrary lists of patrons, but not
deletion. If would be highly beneficial to add patron deletion to the
batch patron modification tool.

Test Plan:
1) Apply all dependencies for this patch
2) Apply this patch
3) Create a list of patrons with the new Patron Lists feature
   a) Include at least one patron owing fines
   b) Include at least one patron with items currently checked out
   c) Include at least one patron not falling into a) or b)
4) Browse to the batch patron modifications tool
5) Select your list from the pulldown, and submit
6) Check the Delete patrons checkbox, the click the submit button
7) You should a list of errors for the patrons with fines or issues
   and a table of patrons that were deleted successfully.
8) Click the link for a deleted patron, you should get a patron not
   found message.

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