source: main/trunk/greenstone2/perllib/plugins/GreenstoneSQLPlugin.pm@ 32589

Last change on this file since 32589 was 32589, checked in by ak19, 5 years ago
  1. SQL db password is not compulsory. 2. Forgot to add the non-compulsory db_port option to the GS SQL Plugs
File size: 22.3 KB
RevLine 
[32536]1###########################################################################
2#
[32542]3# GreenstoneSQLPlugin.pm -- reads into doc_obj from SQL db and docsql.xml
[32536]4# Metadata and/or fulltext are stored in SQL db, the rest may be stored in
5# the docsql .xml files.
6# A component of the Greenstone digital library software
7# from the New Zealand Digital Library Project at the
8# University of Waikato, New Zealand.
9#
10# Copyright (C) 2001 New Zealand Digital Library Project
11#
12# This program is free software; you can redistribute it and/or modify
13# it under the terms of the GNU General Public License as published by
14# the Free Software Foundation; either version 2 of the License, or
15# (at your option) any later version.
16#
17# This program is distributed in the hope that it will be useful,
18# but WITHOUT ANY WARRANTY; without even the implied warranty of
19# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
20# GNU General Public License for more details.
21#
22# You should have received a copy of the GNU General Public License
23# along with this program; if not, write to the Free Software
24# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
25#
26###########################################################################
27
28package GreenstoneSQLPlugin;
29
30
31use strict;
32no strict 'refs'; # allow filehandles to be variables and viceversa
33
34use DBI;
35use docprint; # for new unescape_text() subroutine
36use GreenstoneXMLPlugin;
37use gssql;
38
39
40# TODO:
[32541]41# - Run TODOs here, in Plugout and in gssql.pm by Dr Bainbridge.
[32536]42# - Have not yet tested writing out just meta or just fulltxt to sql db and reading just that
43# back in from the sql db while the remainder is to be read back in from the docsql .xml files.
44
[32563]45# TODO: Add public instructions on using this plugin and its plugout: start with installing mysql binary, changing pwd, running the server (and the client against it for checking: basic cmds like create and drop). Then discuss db name, table names (per coll), db cols and col types, and how the plugout and plugin work.
46# Discuss the plugin/plugout parameters.
[32555]47
[32577]48# TODO, test on windows and mac.
49# Note: if parsing fails (e.g. using wrong plugout like GS XML plugout, which chokes on args intended for SQL plugout) then SQL plugin init would have already been called and done connection, but disconnect would not have been done because SQL plugin disconnect would not have been called upon parse failure.
[32555]50
[32563]51# DONE:
[32583]52# + TODO: For on cancel, add a SIGTERM handler or so to call end()
53# or to explicitly call gs_sql->close_connection if $gs_sql def
54#
[32555]55# + TODO: Incremental delete can't work until GSSQLPlugout has implemented build_mode = incremental
56# (instead of tossing away db on every build)
57# + Ask about docsql naming convention adopted to identify OID. Better way?
58# collection names -> table names: it seems hyphens not allowed. Changed to underscores.
59# + Startup parameters (except removeold/build_mode)
60# + how do we detect we're to do removeold during plugout in import.pl phase
61# + incremental building: where do we need to add code to delete rows from our sql table after
62# incrementally importing a coll with fewer docs (for instance)? What about deleted/modified meta?
63# + Ask if I can assume that all SQL dbs (not just MySQL) will preserve the order of inserted nodes
64# (sections) which in this case had made it easy to reconstruct the doc_obj in memory in the correct order.
65# YES: Otherwise for later db types (drivers), can set order by primary key column and then order by did column
[32563]66# + NOTTODO: when db is not running GLI is paralyzed -> can we set timeout on DBI connection attempt?
67# NOT A PROBLEM: Tested to find DBI connection attempt fails immediately when MySQL server not
68# running. The GLI "paralyzing" incident last time was not because of the gs sql connection code,
69# but because my computer was freezing on-and-off.
70# + "Courier" demo documents in lucene-sql collection: character (degree symbol) not preserved in title. Is this because we encode in utf8 when putting into db and reading back in?
71# Test doc with meta and text like macron in Maori text.
72# + TODO Q: During import, the GS SQL Plugin is called before the GS SQL Plugout with undesirable side
73# effect that if the db doesn't exist, gssql::use_db() fails, as it won't create db.
74# This got fixed when GSSQLPlugin stopped connecting on init().
[32577]75#
76#
77#+ TODO: deal with incremental vs removeold. If docs removed from import folder, then import step
78# won't delete it from archives but buildcol step will. Need to implement this with this database plugin or wherever the actual flow is.
79#
80# + TODO Q: is "reindex" = del from db + add to db?
81# - is this okay for reindexing, or will it need to modify existing values (update table)
82# - if it's okay, what does reindex need to accomplish (and how) if the OID changes because hash id produced is different?
83# - delete is accomplished in GS SQL Plugin, during buildcol.pl. When should reindexing take place?
84# during SQL plugout/import.pl or during plugin? If adding is done by GSSQLPlugout, does it need to
85# be reimplemented in GSSQLPlugin to support the adding portion of reindexing.
86#
87# INCREMENTAL REBUILDING IMPLEMENTED CORRECTLY AND WORKS:
88# Overriding plugins' remove_all() method covered removeold.
89# Overriding plugins' remove_one() method is all I needed to do for reindex and deletion
90# (incremental and non-incremental) to work.
91# but doing all this needed an overhaul of gssql.pm and its use by the GS SQL plugin and plugout.
92# - needed to correct plugin.pm::remove_some() to process all files
93# - and needed to correct GreenstoneSQLPlugin::close_document() to setOID() after all
94# All incremental import and buildcol worked after that:
95# - deleting files and running incr-import and incr-buildcol (= "incr delete"),
96# - deleting files and running incr-import and buildcol (="non-incr delete")
97# - modifying meta and doing an incr rebuild
98# - modifying fulltext and doing an incr rebuild
99# - renaming a file forces a reindex: doc is removed from db and added back in, due to remove_one()
100# - tested CSV file: adding some records, changing some records
101# + CSVPlugin test (collection csvsql)
102# + MetadataCSVPlugin test (modified collection sqltest to have metadata.csv refer to the
103# filenames of sqltest's documents)
104# + shared image test (collection shareimg): if 2 html files reference the same image, the docs
105# are indeed both reindexed if the image is modified (e.g. I replaced the image with another
106# of the same name) which in the GS SQL plugin/plugout case is that the 2 docs are deleted
107# and added in again.
[32555]108
109########################################################################################
110
[32536]111# GreenstoneSQLPlugin inherits from GreenstoneXMLPlugin so that it if meta or fulltext
112# is still written out to doc.xml (docsql .xml), that will be processed as usual,
113# whereas GreenstoneSQLPlugin will process all the rest (full text and/or meta, whichever
114# is written out by GreenstoneSQLPlugout into the SQL db).
115
116
117sub BEGIN {
118 @GreenstoneSQLPlugin::ISA = ('GreenstoneXMLPlugin');
119}
120
121# This plugin must be in the document plugins pipeline IN PLACE OF GreenstoneXMLPlugin
122# So we won't have a process exp conflict here.
[32542]123# The structure of docsql.xml files is identical to doc.xml and the contents are similar except:
124# - since metadata and/or fulltxt are stored in mysql db instead, just XML comments indicating
125# this are left inside docsql.xml within the <Description> (for meta) and/or <Content> (for txt)
126# - the root element Archive now has a docoid attribute: <Archive docoid="OID">
[32536]127sub get_default_process_exp {
128 my $self = shift (@_);
129
[32542]130 return q^(?i)docsql(-\d+)?\.xml$^; # regex based on this method in GreenstoneXMLPlugin
[32536]131}
132
133my $process_mode_list =
134 [ { 'name' => "meta_only",
[32537]135 'desc' => "{GreenstoneSQLPlug.process_mode.meta_only}" },
[32536]136 { 'name' => "text_only",
[32537]137 'desc' => "{GreenstoneSQLPlug.process_mode.text_only}" },
[32536]138 { 'name' => "all",
[32537]139 'desc' => "{GreenstoneSQLPlug.process_mode.all}" } ];
[32536]140
[32582]141my $rollback_on_cancel_list =
142 [ { 'name' => "true",
143 'desc' => "{GreenstoneSQLPlug.rollback_on_cancel}" },
144 { 'name' => "false",
145 'desc' => "{GreenstoneSQLPlug.rollbacl_on_cancel}" } ];
146
[32536]147my $arguments =
148 [ { 'name' => "process_exp",
149 'desc' => "{BaseImporter.process_exp}",
150 'type' => "regexp",
151 'deft' => &get_default_process_exp(),
152 'reqd' => "no" },
[32541]153 { 'name' => "process_mode",
154 'desc' => "{GreenstoneSQLPlug.process_mode}",
155 'type' => "enum",
156 'list' => $process_mode_list,
157 'deft' => "all",
158 'reqd' => "no"},
[32582]159 { 'name' => "rollback_on_cancel",
160 'desc' => "{GreenstoneSQLPlug.rollback_on_cancel}",
161 'type' => "enum",
162 'list' => $rollback_on_cancel_list,
[32586]163 'deft' => "false", # TODO Q: what's the better default? If "true", any memory concerns?
[32582]164 'reqd' => "no",
165 'hiddengli' => "no"},
[32541]166 { 'name' => "db_driver",
167 'desc' => "{GreenstoneSQLPlug.db_driver}",
168 'type' => "string",
169 'deft' => "mysql",
170 'reqd' => "yes"},
171 { 'name' => "db_client_user",
172 'desc' => "{GreenstoneSQLPlug.db_client_user}",
173 'type' => "string",
174 'deft' => "root",
175 'reqd' => "yes"},
176 { 'name' => "db_client_pwd",
177 'desc' => "{GreenstoneSQLPlug.db_client_pwd}",
178 'type' => "string",
179 'deft' => "",
[32589]180 'reqd' => "no"}, # pwd required? NO.
[32541]181 { 'name' => "db_host",
182 'desc' => "{GreenstoneSQLPlug.db_host}",
183 'type' => "string",
184 'deft' => "127.0.0.1",
185 'reqd' => "yes"},
[32589]186 { 'name' => "db_port",
187 'desc' => "{GreenstoneSQLPlug.db_port}",
188 'type' => "string", # NOTE: make this int? No default for port, since it's not a required connection param
189 'reqd' => "no"}
[32536]190 ];
191
192my $options = { 'name' => "GreenstoneSQLPlugin",
193 'desc' => "{GreenstoneSQLPlugin.desc}",
194 'abstract' => "no",
195 'inherits' => "yes",
196 'args' => $arguments };
197
198
[32583]199###### Methods called during buildcol and import #######
[32536]200
201sub new {
202 my ($class) = shift (@_);
203 my ($pluginlist,$inputargs,$hashArgOptLists) = @_;
204 push(@$pluginlist, $class);
205
206 push(@{$hashArgOptLists->{"ArgList"}},@{$arguments});
207 push(@{$hashArgOptLists->{"OptList"}},$options);
208
209 my $self = new GreenstoneXMLPlugin($pluginlist, $inputargs, $hashArgOptLists);
210
211
212 #return bless $self, $class;
213 $self = bless $self, $class;
214 if ($self->{'info_only'}) {
215 # If running pluginfo, we don't need to go further.
216 return $self;
217 }
218
219 # do anything else that needs to be done here when not pluginfo
220
221 return $self;
222}
223
[32583]224# GS SQL Plugin::init() (and deinit()) is called by import.pl and also by buildcol.pl
225# This means it connects and deconnects during import.pl as well. This is okay
226# as removeold, which should drop the collection tables, happens during the import phase,
227# calling GreenstoneSQLPlugin::and therefore also requires a db connection.
228# TODO: Eventually can try moving get_gssql_instance into gssql.pm? That way both GS SQL Plugin
229# and Plugout would be using one connection during import.pl phase when both plugs exist.
[32582]230
[32583]231# Call init() not begin() because there can be multiple plugin passes and begin() called for
232# each pass (one for doc level and another for section level indexing), whereas init() should
233# be called before any and all passes.
234# This way, we can connect to the SQL database once per buildcol run.
235sub init {
236 my ($self) = shift (@_);
237 ##print STDERR "@@@@@@@@@@ INIT CALLED\n";
238
239 $self->SUPER::init(@_); # super (GreenstoneXMLPlugin) will not yet be trying to read from doc.xml (docsql .xml) files in init().
240
241 ####################
[32586]242# print "@@@ SITE NAME: ". $self->{'site'} . "\n" if defined $self->{'site'};
[32583]243# print "@@@ COLL NAME: ". $ENV{'GSDLCOLLECTION'} . "\n";
244
245# print STDERR "@@@@ db_pwd: " . $self->{'db_client_pwd'} . "\n";
246# print STDERR "@@@@ user: " . $self->{'db_client_user'} . "\n";
247# print STDERR "@@@@ db_host: " . $self->{'db_host'} . "\n";
248# print STDERR "@@@@ db_driver: " . $self->{'db_driver'} . "\n";
249 ####################
250
251 # create gssql object.
252 # collection name will be used for naming tables (site name will be used for naming database)
253 my $gs_sql = new gssql({
254 'collection_name' => $ENV{'GSDLCOLLECTION'},
255 'verbosity' => $self->{'verbosity'} || 0
256 });
257
258 # if autocommit is set, there's no rollback support
259 my $autocommit = ($self->{'rollback_on_cancel'} eq "false") ? 1 : 0;
260
261 # try connecting to the mysql db, die if that fails
262 if(!$gs_sql->connect_to_db({
263 'db_driver' => $self->{'db_driver'},
264 'db_client_user' => $self->{'db_client_user'},
265 'db_client_pwd' => $self->{'db_client_pwd'},
266 'db_host' => $self->{'db_host'},
[32589]267 'db_port' => $self->{'db_port'}, # undef by default, can leave as is
[32583]268 'autocommit' => $autocommit
269 })
270 )
271 {
272 # This is fatal for the plugout, let's terminate here
273 # PrintError would already have displayed the warning message on connection fail
274 die("Could not connect to db. Can't proceed.\n");
275 }
276
[32586]277 my $db_name = $self->{'site'} || "greenstone2"; # one database per GS3 site, for GS2 the db is called greenstone2
[32583]278
279 # Attempt to use the db, create it if it doesn't exist (but don't create the tables yet)
280 # Bail if we can't use the database
281 if(!$gs_sql->use_db($db_name)) {
282
283 # This is fatal for the plugout, let's terminate here after disconnecting again
284 # PrintError would already have displayed the warning message on load fail
285 # And on die() perl will call gssql destroy which will ensure a disconnect() from db
286 #$gs_sql->force_disconnect_from_db();
287 die("Could not use db $db_name. Can't proceed.\n");
288 }
289
290
291 # store db handle now that we're connected
292 $self->{'gs_sql'} = $gs_sql;
293}
294
295
296# This method also runs on import.pl if gs_sql has a value. But we just want to run it on buildcol
297# Call deinit() not end() because there can be multiple plugin passes:
298# one for doc level and another for section level indexing
299# and deinit() should be called before all passes
300# This way, we can close the SQL database once per buildcol run.
301sub deinit {
302 my ($self) = shift (@_);
303
304 ##print STDERR "@@@@@@@@@@ GreenstoneSQLPlugin::DEINIT CALLED\n";
305
306 if($self->{'gs_sql'}) {
307
308 # Important to call finished():
309 # it will disconnect from db if this is the last gssql instance,
310 # and it will commit to db before disconnecting if rollbback_on_cancel turned on
311 $self->{'gs_sql'}->finished();
312
313 # Clear gs_sql (setting key to undef has a different meaning from deleting:
314 # undef makes key still exist but its value is unded whereas delete deletes the key)
315 # So all future use has to make the connection again
316 delete $self->{'gs_sql'};
317 }
318
319 $self->SUPER::deinit(@_);
320}
321
322
323
324###### Methods only called during import.pl #####
325
[32563]326# This is called once if removeold is set with import.pl. Most plugins will do
327# nothing but if a plugin does any stuff outside of creating doc obj, then
328# it may need to clear something.
329# In the case of GreenstoneSQL plugs: this is the first time we have a chance
330# to purge the tables of the current collection from the current site's database
331sub remove_all {
332 my $self = shift (@_);
333 my ($pluginfo, $base_dir, $processor, $maxdocs) = @_;
334
[32577]335 $self->SUPER::remove_all(@_);
336
[32563]337 print STDERR " Building with removeold option set, so deleting current collection's tables if they exist\n" if($self->{'verbosity'});
338
[32586]339 # if we're in here, we'd already have run 'use database <site>;' during sub init()
[32563]340 # so we can go ahead and delete the collection's tables
341 my $gs_sql = $self->{'gs_sql'};
342 $gs_sql->delete_collection_tables(); # will delete them if they exist
343
344 # and recreate tables? No. Tables' existence is ensured in GreenstoneSQLPlugout::begin()
345 my $proc_mode = $self->{'process_mode'};
346 if($proc_mode ne "text_only") {
347 $gs_sql->ensure_meta_table_exists();
348 }
349 if($proc_mode ne "meta_only") {
350 $gs_sql->ensure_fulltxt_table_exists();
351 }
[32582]352
[32563]353}
354
[32582]355# This is called during import.pl per document for docs that have been deleted from the
[32563]356# collection. Most plugins will do nothing
357# but if a plugin does any stuff outside of creating doc obj, then it may need
358# to clear something.
359# remove the doc(s) denoted by oids from GS SQL db
360# This takes care of incremental deletes (docs marked D by ArchivesInfPlugin when building
361# incrementally) as well as cases of "Non-icremental Delete", see ArchivesInfPlugin.pm
362sub remove_one {
363 my $self = shift (@_);
364
365 my ($file, $oids, $archivedir) = @_;
366
[32570]367 my $rv = $self->SUPER::remove_one(@_);
368
[32563]369 print STDERR "@@@ IN SQLPLUG::REMOVE_ONE: $file\n";
370
[32570]371 #return undef unless $self->can_process_this_file($file); # NO, DON'T DO THIS (inherited remove_one behaviour) HERE:
[32563]372 # WE DON'T CARE IF IT'S AN IMAGE FILE THAT WAS DELETED.
[32578]373 # WE CARE ABOUT REMOVING THE DOC_OID OF THAT IMAGE FILE FROM THE SQL DB
[32570]374 # SO DON'T RETURN IF CAN'T_PROCESS_THIS_FILE
375
[32563]376
377 my $gs_sql = $self->{'gs_sql'} || return 0; # couldn't make the connection or no db etc
[32577]378
379 print STDERR "*****************************\nAsked to remove_one oid\n***********************\n";
380 print STDERR "Num oids: " . scalar (@$oids) . "\n";
[32563]381
382 my $proc_mode = $self->{'process_mode'};
383 foreach my $oid (@$oids) {
384 if($proc_mode eq "all" || $proc_mode eq "meta_only") {
385 print STDERR "@@@@@@@@ Deleting $oid from meta table\n" if $self->{'verbosity'} > 2;
386 $gs_sql->delete_recs_from_metatable_with_docid($oid);
387 }
388 if($proc_mode eq "all" || $proc_mode eq "text_only") {
389 print STDERR "@@@@@@@@ Deleting $oid from fulltxt table\n" if $self->{'verbosity'} > 2;
390 $gs_sql->delete_recs_from_texttable_with_docid($oid);
391 }
392 }
[32570]393 return $rv;
[32563]394}
395
[32583]396##### Methods called only during buildcol #####
[32563]397
[32542]398sub xml_start_tag {
399 my $self = shift(@_);
400 my ($expat, $element) = @_;
[32536]401
[32542]402 my $outhandle = $self->{'outhandle'};
403
404 $self->{'element'} = $element;
405 if ($element eq "Archive") { # docsql.xml files contain a OID attribute on Archive element
406 # the element's attributes are in %_ as per ReadXMLFile::xml_start_tag() (while $_
407 # contains the tag)
408
409 # Don't access %_{'docoid'} directly: keep getting a warning message to
410 # use $_{'docoid'} for scalar contexts, but %_ is the element's attr hashmap
411 # whereas $_ has the tag info. So we don't want to do $_{'docoid'}.
412 my %attr_hash = %_; # right way, see OAIPlugin.pm
[32555]413 $self->{'doc_oid'} = $attr_hash{'docoid'};
[32542]414 print $outhandle "Extracted OID from docsql.xml: ".$self->{'doc_oid'}."\n"
[32544]415 if $self->{'verbosity'} > 2;
[32542]416
417 }
418 else { # let superclass GreenstoneXMLPlugin continue to process <Section> and <Metadata> elements
419 $self->SUPER::xml_start_tag(@_);
420 }
421}
422
[32555]423# TODO Q: Why are there 4 passes when we're only indexing at doc and section level (2 passes)? What's the dummy pass, why is there a pass for infodb?
[32536]424
[32563]425# We should only ever get here during the buildcol.pl phase
[32536]426# At the end of superclass GreenstoneXMLPlugin.pm's close_document() method,
427# the doc_obj in memory is processed (indexed) and then made undef.
428# So we have to work with doc_obj before superclass close_document() is finished.
429sub close_document {
430 my $self = shift(@_);
[32555]431
[32563]432 ##print STDERR "XXXXXXXXX in SQLPlugin::close_doc()\n";
[32536]433
[32563]434 my $gs_sql = $self->{'gs_sql'};
[32555]435
[32536]436 my $outhandle = $self->{'outhandle'};
[32544]437 my $doc_obj = $self->{'doc_obj'};
[32536]438
[32542]439 my $oid = $self->{'doc_oid'}; # we stored current doc's OID during sub xml_start_tag()
[32555]440 my $proc_mode = $self->{'process_mode'};
441
[32538]442 # For now, we have access to doc_obj (until just before super::close_document() terminates)
[32563]443
[32570]444 # OID parsed of docsql.xml file does need to be set on $doc_obj, as noticed in this case:
445 # when a doc in import is renamed, and you do incremental import, it is marked for reindexing
446 # (reindexing is implemented by this plugin as a delete followed by add into the sql db).
447 # In that case, UNLESS you set the OID at this stage, the old deleted doc id (for the old doc
448 # name) continues to exist in the index at the end of incremental rebuilding if you were to
449 # browse the rebuilt collection by files/titles. So unless you set the OID here, the deleted
450 # doc oids will still be listed in the index.
451 $self->{'doc_obj'}->set_OID($oid);
452
[32563]453 print STDERR " GreenstoneSQLPlugin processing doc $oid (reading into docobj from SQL db)\n"
[32584]454 if $self->{'verbosity'};
[32555]455
[32563]456 if($proc_mode eq "all" || $proc_mode eq "meta_only") {
457 # read in meta for the collection (i.e. select * from <col>_metadata table
[32555]458
[32575]459 my $records = $gs_sql->select_from_metatable_matching_docid($oid, $outhandle);
[32555]460
[32563]461 print $outhandle "----------SQL DB contains meta-----------\n" if $self->{'verbosity'} > 2;
462 # https://www.effectiveperlprogramming.com/2010/07/set-custom-dbi-error-handlers/
[32575]463
464 foreach my $row (@$records) {
465 #print $outhandle "row: @$row\n";
466 my ($primary_key, $did, $sid, $metaname, $metaval) = @$row;
[32555]467
[32563]468 # get rid of the artificial "root" introduced in section id when saving to sql db
469 $sid =~ s@^root@@;
470 $sid = $doc_obj->get_top_section() unless $sid;
471 print $outhandle "### did: $did, sid: |$sid|, meta: $metaname, val: $metaval\n"
[32544]472 if $self->{'verbosity'} > 2;
[32536]473
[32563]474 # TODO: we accessed the db in utf8 mode, so, we can call doc_obj->add_utf8_meta directly:
475 $doc_obj->add_utf8_metadata($sid, $metaname, &docprint::unescape_text($metaval));
[32536]476 }
[32563]477 print $outhandle "----------FIN READING DOC's META FROM SQL DB------------\n"
478 if $self->{'verbosity'} > 2;
479 }
480
481 if($proc_mode eq "all" || $proc_mode eq "text_only") {
482 # read in fulltxt for the collection (i.e. select * from <col>_fulltxt table
[32536]483
[32563]484 my $fulltxt_table = $gs_sql->get_fulltext_table_name();
[32536]485
[32563]486
[32575]487 my $records = $gs_sql->select_from_texttable_matching_docid($oid, $outhandle);
[32563]488
[32575]489
[32563]490 print $outhandle "----------\nSQL DB contains txt entries for-----------\n"
491 if $self->{'verbosity'} > 2;
[32575]492
493 foreach my $row (@$records) {
494 my ($primary_key, $did, $sid, $text) = @$row;
[32555]495
[32563]496 # get rid of the artificial "root" introduced in section id when saving to sql db
497 #$sid =~ s@^root@@;
498 $sid = $doc_obj->get_top_section() if ($sid eq "root");
499 print $outhandle "### did: $did, sid: |$sid|, fulltext: <TXT>\n"
[32544]500 if $self->{'verbosity'} > 2;
[32563]501
502 # TODO - pass by ref?
503 # TODO: we accessed the db in utf8 mode, so, we can call doc_obj->add_utf8_text directly:
[32575]504 my $textref = &docprint::unescape_textref(\$text);
505 $doc_obj->add_utf8_text($sid, $$textref);
[32563]506 }
507 print $outhandle "----------FIN READING DOC's TXT FROM SQL DB------------\n"
508 if $self->{'verbosity'} > 2;
509 }
[32536]510
[32563]511 # done reading into docobj from SQL db
512
[32536]513 # don't forget to clean up on close() in superclass
514 # It will get the doc_obj indexed then make it undef
515 $self->SUPER::close_document(@_);
516}
517
518
[32583]5191;
Note: See TracBrowser for help on using the repository browser.