Problem/Motivation

Redirect entity doesn't have support for entity_language() that is a generic way to access the entity language.

Proposed resolution

Add 'language' property on redirect_entity_info, so it's possible to get the redirect language by using entity_language('redirect', $redirect) function.

CommentFileSizeAuthor
#3 redirect-support_entity_language-2843258-3.patch341 bytesdanilo_formagio
#2 redirect-support_entity_language-2843258-1.patch341 bytesdanilo_formagio

Comments

danilo_formagio created an issue. See original summary.

danilo_formagio’s picture

danilo_formagio commented
StatusFileSize
new redirect-support_entity_language-2843258-1.patch341 bytes
danilo_formagio’s picture

danilo_formagio commented
StatusFileSize
new redirect-support_entity_language-2843258-3.patch341 bytes

The last submitted patch, 2: redirect-support_entity_language-2843258-1.patch, failed testing.

danilo_formagio’s picture

danilo_formagio commented

Status: Needs review » Needs work

The last submitted patch, 3: redirect-support_entity_language-2843258-3.patch, failed testing.

leotorati’s picture

leotorati commented
Title: Implement suppot for entity_language function » Implement support for entity_language function
andre.bonon’s picture

andre.bonon
Primary language Portuguese, Brazil
commented
Status: Needs work » Needs review
andre.bonon’s picture

andre.bonon
Primary language Portuguese, Brazil
commented

Hey Danilo,

I didn't see any impact that it could cause, since the language parameter is optional by default.
It looks good for me.

nbouhid’s picture

nbouhid commented

I don't understand why this wasn't included before. It can't harm.

nbouhid’s picture

nbouhid commented
Status: Needs review » Reviewed & tested by the community
pifagor’s picture

pifagor commented

RTBC

alex_optim’s picture

alex_optim
Location Tallinn
commented

Looks good.

pifagor’s picture

pifagor commented

  • pifagor committed 1529466 on 7.x-2.x 
    Issue #2843258 by danilo_formagio, nbouhid, andre.bonon, pifagor,...
pifagor’s picture

pifagor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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