layoutData, new theme system and Drupal Intergration

Now that layoutData is gone in GalleryEmbed, drupal integration look a bit weird now:

- I guess GalleryCapabilities::set('showSidebar', false) no longer work, it should use GalleryCapabilities::set('showSidebarBlocks', false);
- It seem that GalleryCapabilities::set('showPathbar', false); also no longer work, any workaround on this?
- In Drupal integration, we used to use the layoutData to set the page title as well as building the breadcrumb in Drupal, using the following code:

    if (isset($result['layoutData'])) {
      $urlGenerator =& $GLOBALS['gallery']->getUrlGenerator();
      $breadcrumb = array(l(t('Home'), ''));
      foreach ($result['layoutData']['parents'] as $parent) {
        $breadcrumb[] = '<a href="'.$urlGenerator->generateUrl(array('itemId' => $parent['id'])) .'">'.$parent['title']."</a>";
      }
      drupal_set_breadcrumb($breadcrumb);
      drupal_set_title($result['layoutData']['item']['title']);
    }

Is there any way we can do it in the new theme system?

- Similiar, we also use the layoutData to build the navigation in Drupal, I guess now we can use the $data['showSidebarBlocks'] and extract the navigation block, but we no longer has the fine grain control on the layout because it will have the html code in there as well, and it become pretty ugly because it look a lots different than the rest of the site. (Drupal integration use the following code to generate the navigation

         if (!$result['isDone'] && $result['layoutData']) {
			  $content = "<div class='menu'><ul>";
            $list = array();
            foreach ($result['layoutData']['itemLinks'] as $itemLink) {
              $list[] = '<a href="'.$itemLink['url'] .'">'.$itemLink['text'] ."</a>\n";
            }
            foreach ($result['layoutData']['systemLinks'] as $systemLink) {
              $list[] = '<a href="'.$systemLink['url'] .'">'.$systemLink['text'] ."</a>\n";
            }
			$content = $content . "</ul></div>";
            $block['subject'] = t('gallery');
            $block['content'] = theme('item_list', $list);
	    
          }
        }

Anyway i can extract this information?

BTW, b4 rocks.

Regards,
-SY

thx for the information on the pagetitle.

I did look at the "Embedding and integration" thread before i post this but I could not find the information regarding how to retrieve the layoutData as before ( i guess it is gone). So that's why I post this message.

While I understand that develop a theme specify for your site is the best solution. (I am running a modify version of Matrix since b3). IMHO: I still think that layoutData provide a more flexible option for me to tighyly integrate with the CMS, in this case, Drupal.

Just my 2 cents.

Anyway, go back to create my new theme then

if you want to experiment a little, add
$data['themeData'] =& $template->getVariableByReference('theme');
right after line 313 of main.php

you can use var_dump($data['themeData']); to see what information is available.

lol, was typing the reply when you post this. Thx for the information. I will just do that.

Thanks for the great work in G2.

Regards,
SY
http://www.ocean.hk

i'm curious, how does your code look like?

ok

IMO you could achieve almost the same if we offered GalleryEmbed::getBlockHtml('core.ItemLinks');

and then you'd use CSS to make it look like you want it to, given that the <div> around the itemLinks is selectable etc.

the same goes for breadcrumb and systemlinks.

yes you still have to do this.
please file a feature request. sure i could just add this line to main.php, but IMO we should solve this in a better way, and this will take some time.

no, not there.

add it after these two lines:

$data = $theme->splitHtml($html, $results);
$data['isDone'] = false;

Hi,

I am using a fresh Drupal v4.7.3- and Gallery 2.1.1-Installation. I am using the Hybrid-theme, because it fits very good into the Drupal-Layout and the Hybrid-pictureviewer is pretty handy. I got a php-error logged in drupal pointing to gallery.module line 218 wich starts the following foreach:

if (isset($result['themeData'])) {
   $urlGenerator =& $GLOBALS['gallery']->getUrlGenerator();
   $breadcrumb = array(l(t('Home'), ''));
   foreach ($result['themeData']['parents'] as $parent) {
      $parent_title = $parent['title'];
      // Simple strip of bbcode (italics) 
      $parent_title = str_replace("", "<i>", $parent_title);
      $parent_title = str_replace("", "</i>", $parent_title);
      $breadcrumb[] = '<a href="'.$urlGenerator->generateUrl(
         array('itemId' => $parent['id'])) .'">'.$parent_title."</a>";
   }

The URL this error was produced from, is this one:
http://www.domain.tld/index.php?q=gallery&g2_itemId=123&g2_detail=1

The error is an invalid argument supplied for foreach. Its reason is, that for this URL $result['themeData'] has keys, but no ['parents']. For not removing the $breadcumb = array...-Code I just added another if-block around the for-block:

if( isset($result['themeData']['parents']) ) {
   foreach ($result['themeData']['parents'] as $parent) {
   ...
}

The error is gone and, as far as I can see right now, the code has no further effect on the module. I wrote this in a little naiv way, because I actually don't know neither Drupal-Coding, nor Gallery2-Coding. Just tried some direct PHP-correction for getting my installation to work.

so long,
Felix Lieb
--
YAFA - Yet Another Forum Account

The latest CVS gallery module supports short URLs in the breadcrumb and is 4.7 compatible.