Bug #1472

TBX: 0.7.7, remove text 'path' attribute from TXT+CSV import module or deal with it in appropriate commands

Added by Serge Heiden over 7 years ago. Updated over 7 years ago.

Status:New Start date:09/04/2015
Priority:Urgent Due date:
Assignee:- % Done:

80%
Category:Import Spent time: -
Target version:TXM 0.7.8

Description

Current TXT+CSV import module builds a 'path' property for 'text' structures that one can see in various command results disturbing the user who has never named any source data with that name.

For example after importing the 'voeux-txt' sample corpus without any metadata, the Description command displays:

Propriétés des structures (max 20 valeurs)

p
   id (1) = 0.
s
   n (59) = 22, 23, 24, 25, 26, 27, 28, 29, 3, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43.
text
   id (3) = t0015, t0022, t0036.
   path (1) = "".

That property must not be shown to the user, in:
  • Description command
  • the first page of the Edition of each text
  • Concordance references display choice
  • Sub-corpus selection
  • etc.
    or must not be build.

Solution 1

Remove the 'path' text attribute

History

#1 Updated by Serge Heiden over 7 years ago

  • Subject changed from TBX: 0.7.7, remove text 'path' attribute from TXT+CSV import module to TBX: 0.7.7, remove text 'path' attribute from TXT+CSV import module or deal with it in appropriate commands
  • Description updated (diff)
  • Priority changed from Normal to Urgent

#2 Updated by Serge Heiden over 7 years ago

  • Description updated (diff)

#3 Updated by Matthieu Decorde over 7 years ago

I removed the text@path attribute from the TXT+CSV import (commit 3031)

#4 Updated by Serge Heiden over 7 years ago

Does this mean that the 'path' text structure property was useless?

#5 Updated by Matthieu Decorde over 7 years ago

it has a meaning when the source directory contains sub-directories, anyway the feature is not documented.

#6 Updated by Serge Heiden over 7 years ago

We must then create a ticket & spéc for that feature (to link it to the import process and decide if we publish it or not) AND use the current ticket for another topic concerning internal information.

#7 Updated by Matthieu Decorde over 7 years ago

  • % Done changed from 0 to 50

#8 Updated by Matthieu Decorde over 7 years ago

  • Description updated (diff)

#9 Updated by Matthieu Decorde over 7 years ago

  • Description updated (diff)
  • % Done changed from 50 to 80

Also available in: Atom PDF