inspirehep / inspire-next

The INSPIRE repo.
https://inspirehep.net
GNU General Public License v3.0
59 stars 69 forks source link

Missing title for harvest/submission in holdingpen #3109

Open david-caro opened 6 years ago

david-caro commented 6 years ago

When submitting a new record, using the import with doi 10.3847/1538-4357/834/1/87, the workflow entry that is created in the holdingpen shows no title, but it has actually 3 (you can see in the api).

For example: https://labs.inspirehep.net/api/holdingpen/855695

The data from the api:

"formdata": {
...
    "title": "PRIMUS: ONE- AND TWO-HALO GALACTIC CONFORMITY AT 0.2 <z< 1",
    "title_arXiv": "PRIMUS: One- and Two-Halo Galactic Conformity at $0.2 < z < 1$",
    "title_crossref": "PRIMUS: ONE- AND TWO-HALO GALACTIC CONFORMITY AT 0.2 <z< 1",
    "title_translation": "",
...
},
"metadata": {
...
    "titles": [
        {
            "source": "submitter",
            "title": "PRIMUS: ONE- AND TWO-HALO GALACTIC CONFORMITY AT 0.2 <z< 1"
        },
        {
            "source": "arXiv",
            "title": "PRIMUS: One- and Two-Halo Galactic Conformity at $0.2 < z < 1$"
        },
        {
            "source": "crossref",
            "title": "PRIMUS: ONE- AND TWO-HALO GALACTIC CONFORMITY AT 0.2 <z< 1"
        }
    ]
}

Expected Behavior

It should show the title, even if it's just the first one at least. @michamos might know better what should be there.

Current Behavior

No title is shown.

Steps to Reproduce (for bugs)

1.Submit a new record using the import function for the doi 10.3847/1538-4357/834/1/87

  1. Go to the holdingpen and check the workflow entry.

Screenshot

screenshot from 2018-01-12 19-10-26

david-caro commented 6 years ago

Related to #3111

ksachs commented 5 years ago

another example: https://labs.inspirehep.net/holdingpen/1362501

ksachs commented 5 years ago

Might be caused by (unescaped) '>' or '<'.

StellaCh commented 5 years ago

@harunurhan as the bug ninja, can you investigate whether this is a quick fix or something bigger to add in the backlog?

harunurhan commented 5 years ago

I switched with @drjova for today. But I can have a look tomorrow, if it's not resolved yet.