Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fulltext button not showing #994

Closed
csidirop opened this issue Jul 25, 2023 · 7 comments · Fixed by #999
Closed

Fulltext button not showing #994

csidirop opened this issue Jul 25, 2023 · 7 comments · Fixed by #999

Comments

@csidirop
Copy link
Contributor

Description

Since the main branch got merged into the 4.x the fulltext button and the download links are not shown any more.

Version 4.x from 21 Jul 23: 38b64ca
grafik

Reproduction

  1. Update to newest presentation version
  2. Open a document with fulltext

Expected Behavior

The fulltext button should be shown if there is fulltext available.

Version 4.x from 16 Feb 23: 0c1dc27
grafik

Environment

  • OS version: Debian
  • RDBMS version: MySQL 5.5.5-10.11.2-MariaDB-1:10.11.2+maria~ubu2204
  • Apache Solr version: 8.11.2 (docker)
  • TYPO3 version: 10.4.38-dev
  • PHP version: 7.4.33
@csidirop
Copy link
Contributor Author

csidirop commented Aug 22, 2023

So I digged a bit into that. The fulltext button stopped showing somewhere between the commits 38e759b and 0054abb. Unfortunately lead the first commit to errors (which were handled only in the last commit), so its not clear where exactly the error is.

@beatrycze-volk
Copy link
Collaborator

Thanks for pointing out the commits which introduced the problem.

Could you write down your plugin configuration?

@csidirop
Copy link
Contributor Author

Thanks for your reply!

Which plugin configuration do you mean?

In the meantime I found out, that the error first occurs in this commit 38e759b. If I revert the changes in ToolboxController the fulltext button is shown again. So the other changes have nothing to do with that error.

@beatrycze-volk
Copy link
Collaborator

Which plugin configuration do you mean?

I would need to know how your tools are called in typo script setup.

tools

@csidirop
Copy link
Contributor Author

I did not change any default values on purpose.

dfgviewer/Configuration/TypoScript/Plugins/kitodo.typoscript:

...

# --------------------------------------------------------------------------------------------------------------------
# Tools like imagemanipulation, fulltext and downloads eg.
# --------------------------------------------------------------------------------------------------------------------

# available tools:

# fulltexttool
# annotationtool
# fulltextdownloadtool
# imagedownloadtool
# imagemanipulationtool
# pdfdownloadtool
# searchindocumenttool

plugin.tx_dlf_toolbox < tt_content.list.20.dlf_toolbox
plugin.tx_dlf_toolbox {
    settings {
#        fileGrpsImageDownload = MIN,DEFAULT,MAX
    }
}

plugin.tx_dlf_fulltexttool < plugin.tx_dlf_toolbox
plugin.tx_dlf_fulltexttool {
    settings {
        tools = fulltexttool
        activateFullTextInitially = 0
        fullTextScrollElement = html, body
    }
}

plugin.tx_dlf_imagemanipulationtool < plugin.tx_dlf_toolbox
plugin.tx_dlf_imagemanipulationtool {
    settings {
        tools = imagemanipulationtool
    }
}

plugin.tx_dlf_pdfdownloadtool < plugin.tx_dlf_toolbox
plugin.tx_dlf_pdfdownloadtool {
    settings {
        tools = pdfdownloadtool
    }
}

...

dlf/Configuration/TypoScript/Toolbox/setup.typoscript:

plugin.tx_dlf_fulltexttool {
    settings {
        activateFullTextInitially = 0
        fullTextScrollElement = html, body
        searchHlParameters = tx_dlf[highlight_word]
    }
}

plugin.tx_dlf_searchindocumenttool {
    settings {
        searchUrl =
        documentIdUrlSchema =
        idInputName = tx_dlf[id]
        queryInputName = tx_dlf[query]
        startInputName = tx_dlf[start]
        pageInputName = tx_dlf[page]
        highlightWordInputName = tx_dlf[highlight_word]
        encryptedInputName = tx_dlf[encrypted]
    }
}

@csidirop
Copy link
Contributor Author

csidirop commented Aug 23, 2023

Not sure if this helps, but renderFulltextTool() is called and the if-block at line 181 is executed.

My guess was, that there is some other reference broken here. But I did not found any.

@beatrycze-volk
Copy link
Collaborator

Thank you very much! It actually helped a lot. I think I know what is the problem and I have made a fix for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants