Project:Panels
Version:5.x-2.0-rc1a
Component:Mini panels
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Here is a patch that fixes a few translatable string bugs in panels submodules:

Invalid marker content in panels_mini/panels_mini.module:868
* t(!empty($mini->category)?$mini->category:'Mini panel')

Invalid marker content in panels_page/panels_page.admin.inc:885
* format_plural($args_missing,t('1 argument is missing to generate a valid URL.'),t('%count arguments are missing to generate a valid URL.',
array('%count'=>$args_missing)))
AttachmentSize
panels_2x_rc1a_translatability_bugs.patch2.46 KB

Comments

#1

Status:needs review» needs work

This patch removes lines it should not touch. Please re-roll.

#2

Status:needs work» needs review

What you are talking about? $hidden = $panel_mini->hide_title ? ' (' . t('Title will be hidden') . ')' : ''; is NOT context sensitive (bug) and therefore integrated 3 lines below.

#3

Note: I will commit new POT and German translation file that requires this patch, soon.

#4

This patch needs to be committed first. You cannot speed up the process by committing a translation and translation template for not yet existing strings. Be warned.

#5

This trivial patch is already waiting for one month... please commit

#6

hass, if you ever commit a translation again that requires a patch that is not yet in, I will ask killes and ajk to revoke your CVS access.

I am not kidding. This is not acceptable.

#7

Status:needs review» fixed

Committed, with some modifications b/c some of the problems were already fixed in other commits.

#8

Have you also fixed this in D6? If not it needs a forward port.

#9

Status:fixed» patch (to be ported)

#10

Status:patch (to be ported)» fixed

seems to be done in a i18n patch.

#11

Status:fixed» closed (fixed)

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

nobody click here