Porting D6 Acquia Prosper theme to D7

How have you worked around the fact that Acquia Prosper is dependent on Skinr, and Skinr doesn't yet have a D7 release? I have a D6 site using Acquia Prosper, but I make full use of the Skinr features, so I'd need any D7 version of Acquia Prosper also to use Skinr. I suspect we just have to wait for Skinr to be ready in D7, and then we can all proceed. But I could be wrong - how did you approach this?

The D7 -dev version of Skinr *is* actually working, at least as far as replicating the basic functionality of the Drupal 6 version. We'll be releasing some theme updates soon and clarifying where things are at in a blog post or something. In the meantime, we're trying to help out in the Skinr queue as much as possible.

This does not appear to be calling node-product.tpl.php ?

Subscribing

subscribe

subscribe

Following.

subscribe

Great work!

It is not using node-product.tpl.php for my products, any ideas?

And what function should populate the fields for it? I mean $fusion_uc_image and so on? I can not find it?

Thx

At first blush, this seems to work OK.

Here's #0 + #2 with obsolete LICENSE.txt and $Id$ removed.

This patch is against *master*, which I was assuming was right. If it should be against 6.x-1.x, I'll reroll.

quick fix....

fusion_core template.php

/**
 * Node preprocessing
 */
function fusion_core_preprocess_node(&$vars) {
  // Add to array of handy node classes
  $vars['classes_array'][] = $vars['zebra'];                              // Node is odd or even
  $vars['classes_array'][] = (!$vars['teaser']) ? 'full-node' : '';       // Node is teaser or full-node

  // Add node_top and node_bottom region content
  $vars['node_top'] = theme('blocks', 'node_top');
  $vars['node_bottom'] = theme('blocks', 'node_bottom');
  
  if (module_exists('uc_product') && uc_product_is_product($vars)) {
    
    $vars['fusion_uc_body'] = drupal_render($vars['content']['body']);
    $vars['fusion_uc_image'] = drupal_render($vars['content']['uc_product_image']);
    $vars['fusion_uc_model'] = drupal_render($vars['content']['model']);
    $vars['fusion_uc_display_price'] = drupal_render($vars['content']['display_price']);
    $vars['fusion_uc_add_to_cart'] = drupal_render($vars['content']['add_to_cart']);
    $vars['fusion_uc_sell_price'] = drupal_render($vars['content']['sell_price']);
    $vars['fusion_uc_cost'] = drupal_render($vars['content']['cost']);
    $vars['fusion_uc_weight'] = (!empty($vars['weight'])) ? drupal_render($vars['content']['weight']) : '';   // Hide weight if empty
    if ($vars['fusion_uc_weight'] == '') {
      unset($vars['content']['weight']);
    }
    $dimensions = !empty($vars['height']) && !empty($vars['width']) && !empty($vars['length']);                 // Hide dimensions if empty
    $vars['fusion_uc_dimensions'] = ($dimensions) ? drupal_render($vars['content']['dimensions']) : '';
    if ($vars['fusion_uc_dimensions'] == '') {
      unset($vars['content']['dimensions']);
    }
    $list_price = !empty($vars['list_price']) && $vars['list_price'] > 0;                                     // Hide list price if empty or zero
    $vars['fusion_uc_list_price'] = ($list_price) ? drupal_render($vars['content']['list_price']) : '';
   if ($vars['fusion_uc_list_price'] == '') {
      unset($vars['content']['list_price']);
    }
    $vars['fusion_uc_additional'] = drupal_render($vars['content']);                                        // Render remaining fields
  }
}

and node--product.tpl.php in acquia_prosper

<?php
// $Id
?>

<div id="node-<?php print $node->nid; ?>" class="node <?php print $classes; ?>">
  <div class="inner">
  
    <?php if ($page == 0): ?>
    <h2 class="title"><a href="<?php print $node_url ?>" title="<?php print $title ?>"><?php print $title ?></a></h2>
    <?php endif; ?>
  
    <?php if ($submitted): ?>
    <div class="meta">
      <span class="submitted"><?php print $submitted ?></span>
    </div>
    <?php endif; ?>
  
    <?php if ($node_top && !$teaser): ?>
    <div id="node-top" class="node-top row nested">
      <div id="node-top-inner" class="node-top-inner inner">
        <?php print $node_top; ?>
      </div><!-- /node-top-inner -->
    </div><!-- /node-top -->
    <?php endif; ?>

    <div id="product-group" class="product-group">
      <div class="images">
        <?php print $fusion_uc_image; ?>
      </div><!-- /images -->

      <div class="content">
        <div id="content-body">
          <?php print $fusion_uc_body; ?>
        </div>

        <?php $product_details = $fusion_uc_weight || $fusion_uc_dimensions || $fusion_uc_list_price || $fusion_uc_sell_price || $fusion_uc_model || $fusion_uc_cost; ?>
        <div id="product-details" class="clear<?php if (!$product_details): ?> field-group-empty<?php endif; ?>">
          <div id="field-group">
            <?php print $fusion_uc_weight; ?>
            <?php print $fusion_uc_dimensions; ?>
            <?php print $fusion_uc_model; ?>
            <?php print $fusion_uc_list_price; ?>
            <?php print $fusion_uc_sell_price; ?>
            <?php print $fusion_uc_cost; ?>
          </div>

          <div id="price-group">
            <?php print $fusion_uc_display_price; ?>
            <?php print $fusion_uc_add_to_cart; ?>
          </div>
        </div><!-- /product-details -->

        <?php if ($fusion_uc_additional && !$teaser): ?>
        <div id="product-additional" class="clear product-additional">
          <?php print $fusion_uc_additional; ?>
        </div>
        <?php endif; ?>

      </div><!-- /content -->
    </div><!-- /product-group -->
  </div><!-- /inner -->


  <?php if ($node_bottom && !$teaser): ?>
  <div id="node-bottom" class="node-bottom row nested">
    <div id="node-bottom-inner" class="node-bottom-inner inner">
      <?php print $node_bottom; ?>
    </div><!-- /node-bottom-inner -->
  </div><!-- /node-bottom -->
  <?php endif; ?>
</div><!-- /node-<?php print $node->nid; ?> -->

@bwynants, does this relate to #11? If it's a competing patch, could you roll it as a patch to help the maintainers?

@rfay yes it relates to #11 and no I do not make an official patch for 2 reasons:

1) This is currently not an official Acquia Prosper release, but just a download in the issue queue

2) I haven't yet created a patch now drupal is using gitHub. I have to sort that out but have no time now...

I did report the fusion_core fix on the fusion_core issue queue so that they can get that in if they want.

Just want to help out and expect someone to take it from here, sorry for now.

Regards

@bwynants : Thanks for the work. I tried out the changes in template.php and node--product.tpl.php file with just one modification. Instead of making the changes in fusion_core template file, I added the preprocess_node function in acquia_prosper drupal 7 port template file. And changed all the variable from fusion_uc--- to acquia_prosper_uc---. in the acquia_prosper D7 theme. I am attaching the files below.
Thanks for the work. It really helped.

Thanks,
Prajakta.

Here is #15 rerolled as a patch (and with minor changes removing some obsolete blank lines and a couple of still-hanging-around $Id$.

I note that all of these patches don't show comment bodies, so settings to needs work.

I note also that we now have both node-product.tpl.php AND node--product.tpl.php

Thanks everyone! We're working on this now, now that the Fusion alpha is out. Will have an initial version committed this week.

@rfay node-product.tpl.php is not used in D7, we only need node--product.tpl.php

subscribing

there is a dev version for Drupal 7 out. it does not have skinr or ubercart features in this release. skinr is under heavy development at this time.

Hey everyone,

There's been a lot of talk about eventual ubercart features (which are not yet in the release), but I haven't seen any discussion of Drupal Commerce styling in the D7 version. Will Drupal Commerce features be implemented along with Ubercart features?

Looking forward to testing the new release,
Ben

subscribing

Looks good so far. Haven't found any major issues. why isn't this on the main project page as a d7 dev release?

there is a D7 dev version on main project page for testing and review

@goody815 but it does not have the ubercart features so it's not useful

Hi bwynants,

We are going to be adding Drupal Commerce features for the D7 version of Acquia Prosper. It still should work with ubercart. It just won't have the extra ubercart styling preset as in the D6 version

This thread is very useful. I installed Acquia Prosper and Fusion and get the following error.

Notice: Undefined variable: node_top in include() (line 18 of /home/clients/websites/w_iyo702/public_html/iyo702/sites/all/themes/acquia_prosper/node.tpl.php).
Notice: Undefined variable: node_bottom in include() (line 53 of /home/clients/websites/w_iyo702/public_html/iyo702/sites/all/themes/acquia_prosper/node.tpl.php).

Since I am not real adept at php or coding or applying a patch but can modify the .tpl file. I am not clear on whether the info above regarding the patch takes care of this.

If someone can help I'd really appreciate it.

Thanks,
Mildred

I upgraded my site from Drupal 6.30 to 7.26 and upgraded Acquia Prosper theme as well. Everything seems working fine except pager in views. a block view of user's blog posts is missing a pager which was there in prevoius version.