Currently the extensions list doesn't include the new extensions for MS office documents (.docx, .xlsx, .pptx) which have been the defaults since 2007.

Additionally, not sure if they should be included but a number of extra extensions were added in Office 2007 for thing such as 'macro-enabled document'. The details are here.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

PQ’s picture

PQ’s picture

Oops... sorry, this time with a non-blank patch

PQ’s picture

Status: Active » Needs review
hass’s picture

Status: Needs review » Needs work

Thanks for sharing a patch. We should also add an update hook for this and upgrade all users using the default extension list and warn the users not using the default list about the changes. Let's also add the other extensions with macros/binary while we are already changing it.

hass’s picture

Will you fix the outstanding bugs or should we close the case?

PQ’s picture

Sorry, took a while to get back to this. I've added the extra extensions and the update hook.

hass’s picture

Status: Needs work » Needs review

Thanks

Status: Needs review » Needs work

The last submitted patch, 6: googleanalytics-track-ms-open-xml-downloads-2323215-6.patch, failed testing.

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: googleanalytics-track-ms-open-xml-downloads-2323215-6.patch, failed testing.

PQ’s picture

Not sure what's happening with the php filter tests... This patch really shouldn't affect them I'd have thought.

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: googleanalytics-track-ms-open-xml-downloads-2323215-6.patch, failed testing.

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: googleanalytics-track-ms-open-xml-downloads-2323215-6.patch, failed testing.

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: googleanalytics-track-ms-open-xml-downloads-2323215-6.patch, failed testing.

hass’s picture

There is at least one bug in the JS regex with dot[xm]?.

Status: Needs review » Needs work

The last submitted patch, 18: Issue-2323215-Support-modern-MS-office-documents-doc.patch, failed testing.

hass’s picture

Status: Needs work » Needs review
FileSize
3 KB

Field is only 255, but string is 281. Increased to 500.

hass’s picture

Version: 7.x-2.x-dev » 6.x-4.x-dev
FileSize
3.03 KB
hass’s picture

Fixed update hook bugs

  • hass committed dd99000 on 6.x-4.x
    Issue #2323215: Support modern MS office documents (.docx, .xlsx, .pptx)
    

  • hass committed a1047df on
    Issue #2323215: Support modern MS office documents (.docx, .xlsx, .pptx)
    

  • hass committed ce718d0 on 8.x-2.x
    Issue #2323215: Support modern MS office documents (.docx, .xlsx, .pptx)
    
hass’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.