[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 #20 from Jonathan Druart  ---
Hello Nora,

Thanks for sharing your experience.
What I suggested to Kyle is compatible with what you describe.
If we can choose a patron list in both the modification and the deletion tools,
I don't see any problem.

Kyle, maybe the best way to do could be to add a link to jump from the patron
list module to the modification or deletion tool. This way, when the patron
list is created, the librarian can directly modify or delete the patrons from
the list.

Let me know if I missed something or if it does not make sense.

-- 
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  ---
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-12-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

--- Comment #18 from Jonathan Druart  ---
(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 #17 from Kyle M Hall  ---
(In reply to Jonathan Druart from comment #15)
> Kyle,
> Why don't you use the Batch patron deletion? Maybe we could have a new entry
> "Delete patrons in the list ".
> Don't you think it is not consistent to allow to delete patron in 2
> different places?

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.

If you consider this change in behavior a QA deal breaker, please let me know
by setting this bug to 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-12-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Jonathan Druart  changed:

   What|Removed |Added

 Status|Signed Off  |In Discussion

--- Comment #16 from Jonathan Druart  ---
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-11-12 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

--- Comment #15 from Jonathan Druart  ---
Kyle,
Why don't you use the Batch patron deletion? Maybe we could have a new entry
"Delete patrons in the list ".
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  changed:

   What|Removed |Added

  Attachment #21446|0   |1
is obsolete||

--- Comment #14 from Kyle M Hall  ---
Created attachment 22838
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22838&action=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-11-08 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #21081|0   |1
is obsolete||

--- Comment #13 from Kyle M Hall  ---
Created attachment 22837
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22837&action=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 

-- 
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  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-09-28 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Katrin Fischer  changed:

   What|Removed |Added

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

--- Comment #12 from Katrin Fischer  ---
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...
:41: trailing whitespace.
Cannot delete patron 
:49: trailing whitespace.
Cannot delete patron 
: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  changed:

   What|Removed |Added

  Attachment #21445|0   |1
is obsolete||

--- Comment #11 from Kyle M Hall  ---
Created attachment 21446
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21446&action=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  changed:

   What|Removed |Added

  Attachment #21444|0   |1
is obsolete||

--- Comment #10 from Kyle M Hall  ---
Created attachment 21445
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21445&action=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

--- Comment #9 from Kyle M Hall  ---
Created attachment 21444
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21444&action=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  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-19 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Jonathan Druart  changed:

   What|Removed |Added

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

--- Comment #8 from Jonathan Druart  ---
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

Kyle M Hall  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-13 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Melissa Lefebvre  changed:

   What|Removed |Added

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

--- Comment #7 from Melissa Lefebvre  ---
Created attachment 21081
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21081&action=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

Melissa Lefebvre  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-09 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Nora Blake  changed:

   What|Removed |Added

 CC||nbl...@masslibsystem.org

--- Comment #6 from Nora Blake  ---
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  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  ---
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.html&bug=10612&attachment=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  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

--- Comment #4 from Kyle M Hall  ---
(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  changed:

   What|Removed |Added

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

--- Comment #3 from Liz Rea  ---
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  changed:

   What|Removed |Added

  Attachment #19766|0   |1
is obsolete||

--- Comment #2 from Kyle M Hall  ---
Created attachment 19767
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19767&action=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

--- Comment #1 from Kyle M Hall  ---
Created attachment 19766
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19766&action=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  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  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

Kyle M Hall  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/