[PATCH] missing header in Layers table [#810732]

Comment	File	Size	Author
#8	LayersTypeHeader2.patch	373 bytes	strk

	LayersTypeHeader.patch	455 bytes	strk

Comment #1

tmcw CreditAttribution: tmcw commented 27 May 2010 at 12:18

Looks like you're behind head; can you try updating? The current code will add a header - this patch adds the type header right under where it already adds a header.

Log in or register to post comments

Comment #2

strk CreditAttribution: strk commented 27 May 2010 at 12:30

I'm up-to-date on the DRUPAL-6--2, should I track HEAD instead ?

Log in or register to post comments

Comment #3

strk CreditAttribution: strk commented 27 May 2010 at 12:31

My patch is against r1.1.2.34

Log in or register to post comments

Comment #4

tmcw CreditAttribution: tmcw commented 27 May 2010 at 12:33

It's all being tracked in DRUPAL-6--2, but isn't the logic on line 382 specifying a type header for object lists that are layers? Try reverting your patch and clearing caches? This is working on my side, and this patch will add headers for styles, not layers or presets.

Log in or register to post comments

Comment #5

strk CreditAttribution: strk commented 27 May 2010 at 12:45

Doh, you're right, it's the wrong block.
Still, reverting my patch drops the Type from the Layers tab too
so must be something with line 686 failing

if ($type == 'layers') {

I think I did drop drupal cache (if 'drush cc' is enough for that).
I surely did drop the browser one.

Log in or register to post comments

Comment #6

strk CreditAttribution: strk commented 27 May 2010 at 12:48

Actually, the block is entered correctly. Maybe I'm looking at the wrong file ?
The problem is on the screen shown with path 'admin/build/openlayers/layers'