When translation is disabled for a content type, in some cases language can be different from 'LANGUAGE_NONE' so additional check is needed

CommentFileSizeAuthor
#2 check_if_translation_enabled-2562497.patch1.17 KBroshkovanv

Comments

roshkovanv created an issue. See original summary.

roshkovanv’s picture

roshkovanv commented
Status: Active » Needs review
StatusFileSize
new check_if_translation_enabled-2562497.patch1.17 KB

Status: Needs review » Needs work

The last submitted patch, 2: check_if_translation_enabled-2562497.patch, failed testing.

rodrigoeg’s picture

rodrigoeg commented

Thanks for this patch @roshkovanv. I will have to review the simple tests to consider this scenario, because the tests are breaking with this patch, but I did a quick review and I think your patch is ok to be merged in the module.

  • rodrigoeg committed 0d98f1a on 7.x-1.x 
    Issue #2562497: Access restricted for content types with disabled...
  • rodrigoeg committed 7b4bc13 on 7.x-1.x authored by roshkovanv
    Issue #2562497 by roshkovanv: Access restricted for content types with...
rodrigoeg’s picture

rodrigoeg commented
Status: Needs work » Fixed

Hi roshkovanv, the changes have been commited with your user as the author. Thanks for your contribution.

Status: Fixed » Needs review

rodrigoeg queued 2: check_if_translation_enabled-2562497.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: check_if_translation_enabled-2562497.patch, failed testing.

rodrigoeg’s picture

rodrigoeg commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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