? Success

User tests: Successful: Unsuccessful:

avatar level420
level420
5 May 2014

When indexing content via a finder plugin which may have multiple mime types (e.g. document management systems, pdf, doc, zip etc.), adding a new mime type will work, but if another document with the same mime type exists, consecutive getting the type id results in a duplicate insert in table #__finder_types.
This is due to that the static method variable $types is initialized on the first call of the method and does not take in account mime types inserted after initialization.

The fix adds the newly inserted mime type into $types hash, so that calling this method againg with the same mimetype results in a positive lookup within the hash.

avatar level420 level420 - open - 5 May 2014
avatar level420
level420 - comment - 5 May 2014

This is for our joomla 2.5.20 installations, but I suspect that this is also true for master.
Sorry If I've made the pull request for the wrong branch.

avatar infograf768
infograf768 - comment - 20 Jul 2014

This has been done towards staging and as it passes ok, it imho means it would also be necessary here.
remains for the pacth to be tested and a similar one done towards 2.5.x

avatar rvbgnu
rvbgnu - comment - 26 Jul 2014

Will need clear testing instructions, please.

avatar nicksavov nicksavov - change - 21 Aug 2014
Title
[#33700] Fix for "duplicate key on insert detected" errors in FinderIndexerHelper::addContentType
[#33700] Fix for "duplicate key on insert detected" errors in FinderIndexerHelper::addContentType
Labels Removed: ?
avatar brianteeman
brianteeman - comment - 22 Aug 2014

Thank you for your contribution and participation in the bug tracking process. As there has been no response to the request for testing information I am closing this report at this time.

We have now moved our bug tracking process to http://issues.joomla.org/ so if you have further information that would enable this issue to be reopened please update the issue at http://issues.joomla.org/tracker/joomla-cms/3558 and it can be reopened.

avatar brianteeman brianteeman - change - 22 Aug 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-08-22 10:36:48
avatar brianteeman brianteeman - close - 22 Aug 2014

Add a Comment

Login with GitHub to post a comment