Bugzilla – Attachment 104179 Details for
Bug 22544
Move C4:NewsChannels to Koha namespace
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 22544: Move upd_opac_new to Koha namespace
Bug-22544-Move-updopacnew-to-Koha-namespace.patch (text/plain), 3.88 KB, created by
Josef Moravec
on 2020-05-02 09:05:43 UTC
(
hide
)
Description:
Bug 22544: Move upd_opac_new to Koha namespace
Filename:
MIME Type:
Creator:
Josef Moravec
Created:
2020-05-02 09:05:43 UTC
Size:
3.88 KB
patch
obsolete
>From 091a65811cf577e28e377036334638726442e5bc Mon Sep 17 00:00:00 2001 >From: Josef Moravec <josef.moravec@gmail.com> >Date: Tue, 19 Mar 2019 10:30:27 +0000 >Subject: [PATCH] Bug 22544: Move upd_opac_new to Koha namespace > >Signed-off-by: David Nind <david@davidnind.com> >--- > C4/NewsChannels.pm | 40 +---------------------------------- > t/db_dependent/NewsChannels.t | 12 +---------- > tools/koha-news.pl | 10 ++++----- > 3 files changed, 7 insertions(+), 55 deletions(-) > >diff --git a/C4/NewsChannels.pm b/C4/NewsChannels.pm >index a05733ddd6..6ead4104c0 100644 >--- a/C4/NewsChannels.pm >+++ b/C4/NewsChannels.pm >@@ -28,7 +28,7 @@ BEGIN { > @ISA = qw(Exporter); > @EXPORT = qw( > &GetNewsToDisplay >- &add_opac_new &upd_opac_new >+ &add_opac_new > ); > } > >@@ -71,44 +71,6 @@ sub add_opac_new { > return $retval; > } > >-=head2 upd_opac_new >- >- $retval = upd_opac_new($hashref); >- >- $hashref should contains all the fields found in opac_news, >- including idnew, since it is the key for the SQL UPDATE. >- >-=cut >- >-sub upd_opac_new { >- my ($href_entry) = @_; >- my $retval = 0; >- >- if ($href_entry) { >- # take the keys of hash entry and make a list, but... >- my @fields = keys %{$href_entry}; >- my @values; >- $#values = -1; >- my $field_string = q{}; >- foreach my $field_name (@fields) { >- # exclude idnew >- if ( $field_name ne 'idnew' ) { >- $field_string = $field_string . "$field_name = ?,"; >- push @values,$href_entry->{$field_name}; >- } >- } >- # put idnew at the end, so we know which record to update >- push @values,$href_entry->{'idnew'}; >- chop $field_string; # remove that excess , >- >- my $dbh = C4::Context->dbh; >- my $sth = $dbh->prepare("UPDATE opac_news SET $field_string WHERE idnew = ?;"); >- $sth->execute(@values); >- $retval = 1; >- } >- return $retval; >-} >- > =head2 GetNewsToDisplay > > $news = &GetNewsToDisplay($lang,$branch); >diff --git a/t/db_dependent/NewsChannels.t b/t/db_dependent/NewsChannels.t >index f14d48939e..0a66177a3e 100644 >--- a/t/db_dependent/NewsChannels.t >+++ b/t/db_dependent/NewsChannels.t >@@ -6,7 +6,7 @@ use Koha::DateUtils; > use Koha::Libraries; > use Koha::News; > >-use Test::More tests => 9; >+use Test::More tests => 7; > > BEGIN { > use_ok('C4::NewsChannels'); >@@ -125,16 +125,6 @@ $query = > q{ SELECT idnew from opac_news WHERE timestamp='2000-01-02'; }; > my ( $idnew3 ) = $dbh->selectrow_array( $query ); > >-# Test upd_opac_new >-$rv = upd_opac_new(); # intentionally bad parmeters >-is( $rv, 0, 'Correctly failed on no parameter!' ); >- >-$new2 = '<p>Update! There is no news!</p>'; >-$href_entry2->{content} = $new2; >-$href_entry2->{idnew} = $idnew2; >-$rv = upd_opac_new($href_entry2); >-is( $rv, 1, 'Successfully updated second dummy news item!' ); >- > # Test GetNewsToDisplay > my ( $opac_news_count, $arrayref_opac_news ) = GetNewsToDisplay( q{}, 'LIB1' ); > ok( $opac_news_count >= 2, 'Successfully tested GetNewsToDisplay for LIB1!' ); >diff --git a/tools/koha-news.pl b/tools/koha-news.pl >index c7b590d209..fa912b6fbb 100755 >--- a/tools/koha-news.pl >+++ b/tools/koha-news.pl >@@ -123,9 +123,9 @@ elsif ( $op eq 'add' ) { > } > } > elsif ( $op eq 'edit' ) { >- upd_opac_new( >- { >- idnew => $id, >+ my $news_item = Koha::News->find( $id ); >+ if ( $news_item ) { >+ $news_item->set({ > title => $title, > content => $content, > lang => $lang, >@@ -133,8 +133,8 @@ elsif ( $op eq 'edit' ) { > timestamp => $timestamp, > number => $number, > branchcode => $branchcode, >- } >- ); >+ })->store; >+ } > print $cgi->redirect("/cgi-bin/koha/tools/koha-news.pl"); > } > elsif ( $op eq 'del' ) { >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 22544
:
86772
|
86773
|
86774
|
86775
|
86776
|
86777
|
86780
|
86781
|
90103
|
90104
|
90105
|
90106
|
90107
|
90108
|
90109
|
90110
|
92005
|
92006
|
92007
|
92008
|
92009
|
92010
|
92011
|
92012
|
95613
|
95614
|
95615
|
95627
|
95628
|
95629
|
95630
|
95631
|
95632
|
95633
|
95634
|
99849
|
99850
|
99851
|
99852
|
99853
|
99854
|
99855
|
99856
|
99857
|
104176
|
104177
|
104178
|
104179
|
104180
|
104181
|
104182
|
104183
|
104184
|
111012
|
111013
|
111014
|
111015
|
111016
|
111017
|
111018
|
111019
|
111020
|
111021
|
111022
|
111023
|
115135
|
115136
|
115137
|
115138
|
115139
|
115140
|
115141
|
115142
|
115143
|
115144
|
115145
|
115146
|
115218
|
115296
|
115297
|
115298
|
115299
|
116933
|
116934
|
116935
|
117479
|
117480
|
117481
|
117482
|
117483
|
117484
|
117485
|
117486
|
117487
|
117488
|
117489
|
117490
|
117491
|
117492
|
117493
|
117494
|
117495
|
117496
|
117497
|
118054
|
118055
|
118056
|
118057
|
118058
|
118059
|
118060
|
118061
|
118062
|
118063
|
118064
|
118065
|
118066
|
118067
|
118068
|
118069
|
118070
|
118071
|
118072
|
118073
|
118087
|
118088
|
118089
|
118090
|
118091
|
118092
|
118093
|
118094
|
118095
|
118096
|
118097
|
118098
|
118099
|
118100
|
118101
|
118102
|
118103
|
118104
|
118105
|
118106
|
120377
|
120378
|
120379
|
120380
|
120381
|
120382
|
120383
|
120384
|
120385
|
120386
|
120387
|
120388
|
120389
|
120390
|
120391
|
120392
|
120393
|
120394
|
120395
|
120396
|
122396
|
122397
|
122398
|
122399
|
122400
|
122401
|
122402
|
122403
|
122404
|
122405
|
122406
|
122407
|
122408
|
122409
|
122410
|
122411
|
122412
|
122413
|
122414
|
122415