X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=C4%2FMaintainance.pm;h=8726ef91f1da39c8b0e556c4f27430261d3a0d91;hb=9c5e40e4923179bf0b2b630d3e09797dc4c1fdb0;hp=107361c8920f16c01312072a85d7d6a986633588;hpb=ec94689218fa2cbf0360fa1a731bac4b0e028b63;p=koha.git

diff --git a/C4/Maintainance.pm b/C4/Maintainance.pm
index 107361c892..8726ef91f1 100644
--- a/C4/Maintainance.pm
+++ b/C4/Maintainance.pm
@@ -16,11 +16,12 @@ package C4::Maintainance; #assumes C4/Maintainance
 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
 # A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
 use strict;
+#use warnings; FIXME - Bug 2505
 use C4::Context;
 
 require Exporter;
@@ -28,7 +29,7 @@ require Exporter;
 use vars qw($VERSION @ISA @EXPORT);
 
 # set the version for version checking
-$VERSION = 0.01;
+$VERSION = 3.07.00.049;
 
 =head1 NAME
 
@@ -46,15 +47,13 @@ miscategorized items, etc.
 
 =head1 FUNCTIONS
 
-=over 2
-
 =cut
 
 @ISA = qw(Exporter);
-@EXPORT = qw(&listsubjects &updatesub &shiftgroup &deletedbib &undeletebib
+@EXPORT = qw(&listsubjects &shiftgroup &deletedbib &undeletebib
 &updatetype &logaction);
 
-=item listsubjects
+=head2 listsubjects
 
   ($count, $results) = &listsubjects($subject, $n, $offset);
 
@@ -73,6 +72,7 @@ C<&listsubject> returns up to C<$n> items, starting at C<$offset>. If
 C<$n> is 0, it will return all matching subjects.
 
 =cut
+
 #'
 # FIXME - This API is bogus. The way it's currently used, it should
 # just return a list of strings.
@@ -100,24 +100,7 @@ sub listsubjects {
   return($i,\@results);
 }
 
-=item updatesub
-
-  &updatesub($newsubject, $oldsubject);
-
-Renames a subject from C<$oldsubject> to C<$newsubject> in the
-bibliosubject table of the Koha database.
-
-=cut
-#'
-sub updatesub{
-  my ($sub,$oldsub)=@_;
-  my $dbh = C4::Context->dbh;
-  my $sth=$dbh->prepare("update bibliosubject set subject=? where subject=?");
-  $sth->execute($sub,$oldsub);
-  $sth->finish;
-}
-
-=item shiftgroup
+=head2 shiftgroup
 
   &shiftgroup($biblionumber, $biblioitemnumber);
 
@@ -126,19 +109,20 @@ C<$biblioitemnumber> is the number of the biblioitem to change.
 C<$biblionumber> is the biblionumber to associate it with.
 
 =cut
+
 #'
 sub shiftgroup{
-  my ($bib,$bi)=@_;
+  my ($biblionumber,$bi)=@_;
   my $dbh = C4::Context->dbh;
   my $sth=$dbh->prepare("update biblioitems set biblionumber=? where biblioitemnumber=?");
-  $sth->execute($bib,$bi);
+  $sth->execute($biblionumber,$bi);
   $sth->finish;
   $sth=$dbh->prepare("update items set biblionumber=? where biblioitemnumber=?");
-  $sth->execute($bib,$bi);
+  $sth->execute($biblionumber,$bi);
   $sth->finish;
 }
 
-=item deletedbib
+=head2 deletedbib
 
   ($count, $results) = &deletedbib($title);
 
@@ -150,6 +134,7 @@ the fields of the deletedbiblio table in the Koha database. C<$count>
 is the number of elements in C<$results>.
 
 =cut
+
 #'
 sub deletedbib{
   my ($title)=@_;
@@ -166,7 +151,7 @@ sub deletedbib{
   return($i,\@results);
 }
 
-=item undeletebib
+=head2 undeletebib
 
   &undeletebib($biblionumber);
 
@@ -175,30 +160,32 @@ biblionumber in the deletedbiblio table of the Koha database, and
 moves its entry to the biblio table.
 
 =cut
+
 #'
 sub undeletebib{
-  my ($bib)=@_;
+  my ($biblionumber)=@_;
   my $dbh = C4::Context->dbh;
   my $sth=$dbh->prepare("select * from deletedbiblio where biblionumber=?");
-  $sth->execute($bib);
+  $sth->execute($biblionumber);
   if (my @data=$sth->fetchrow_array){
     $sth->finish;
     # FIXME - Doesn't this keep the same biblionumber? Isn't this
     # forbidden by the definition of 'biblio'? Or doesn't it matter?
-    my $query="Insert into biblio values (";
-    $query .= ("?," x $#data);
+    my $query="INSERT INTO biblio VALUES (";
+   my $count = @data;
+    $query .= ("?," x $count);
     $query=~ s/\,$/\)/;
     #   print $query;
     $sth=$dbh->prepare($query);
     $sth->execute(@data);
     $sth->finish;
   }
-  $sth=$dbh->prepare("Delete from deletedbiblio where biblionumber=?");
-  $sth->execute($bib);
+  $sth=$dbh->prepare("DELETE FROM deletedbiblio WHERE biblionumber=?");
+  $sth->execute($biblionumber);
   $sth->finish;
 }
 
-=item updatetype
+=head2 updatetype
 
   &updatetype($biblioitemnumber, $itemtype);
 
@@ -206,6 +193,7 @@ Changes the type of the item with the given biblioitemnumber to be
 C<$itemtype>.
 
 =cut
+
 #'
 sub updatetype{
   my ($bi,$type)=@_;
@@ -215,31 +203,13 @@ sub updatetype{
   $sth->finish;
 }
 
-=item logaction
-
-  &logaction($usernumber, $modulename, $actionname, $infos);
-
-Adds a record into action_logs table to report the different changes upon the database
-
-=cut
-#'
-sub logaction{
-  my ($usernumber,$modulename, $actionname, $infos)=@_;
-  my $dbh = C4::Context->dbh;
-	my $sth=$dbh->prepare("Insert into action_logs (timestamp,user,module,action,info) values (now(),?,?,?,?)");
-	$sth->execute($usernumber,$modulename,$actionname,$infos);
-	$sth->finish;
-}
-
 END { }       # module clean-up code here (global destructor)
 
 1;
 __END__
 
-=back
-
 =head1 AUTHOR
 
-Koha Developement team <info@koha.org>
+Koha Development Team <http://koha-community.org/>
 
 =cut