should be either full, or teaser, patch supplied

CommentFileSizeAuthor
#3 845170-missing-third-argument.patch465 bytesjp.stacey
field_attach_arg.patch472 bytesOzeuss

Comments

joachim’s picture

joachim commented

Good catch!

Looks good but won't apply after current work.

Will look at this later today.

joachim’s picture

joachim commented

Though this patch is assuming we have a view type called 'full'....

Are view types defined anywhere?

jp.stacey’s picture

jp.stacey commented
StatusFileSize
new 845170-missing-third-argument.patch465 bytes

Great patch - tested and it works fine.

Re-roll attached following #631192: type/id/name/label -- standardize?, so it should probably be tested again. Note that this patch was made against git HEAD: I can't see any other reason for the line numbers being different.

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Fixed

thanks, committed.

Status: Fixed » Closed (fixed)

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