Book Reviews 1.0.1

[Anthony Parsons]

I found an issue in template bookreviews_view_item.

<h3 class="reviewTitle"><span itemprop="name">{$review.review_title}</h3>

As you can see, it is missing its closing span.

Also, the list item reviews as a whole are also not being cleared, so if you have something being entered in the page_container template under content, on the review pages with reviews posted, it screws up due to the floats not being cleared.

 

[Jon W]

Waindigo updated Book Reviews by Waindigo with a new update entry:

Version 1.0.1 released

Performance improvements:

• Phrases for tab title and book types now cached to reduce database queries.
• Added missing html tag in bookreviews_review_item template.
• All tables now stored as InnoDb type with UTF8 encoding.

Read the rest of this update entry...

 

[Jon W]

Oh... also found you hadn't cached your frontend phrases globally, thus adding a query for each phrase as required to the front-end page loading.

I found an issue in template bookreviews_view_item.

<h3 class="reviewTitle"><span itemprop="name">{$review.review_title}</h3>

As you can see, it is missing its closing span.

Also, the list item reviews as a whole are also not being cleared, so if you have something being entered in the page_container template under content, on the review pages with reviews posted, it screws up due to the floats not being cleared.

Thanks for pointing these out Anthony. I have updated the add-on. I hope Rob doesn't mind.

 

[intradox]

Great addon! No rel use for me but would love a game review addon so I might tinker with this and see what I can do.

 

[RedSpades]

Great addon! No rel use for me but would love a game review addon so I might tinker with this and see what I can do.

No point tickering, really.

The review sections are solely for books and some of the fields ask for book information.

Still waiting for this mod to be for a more general use and not just for books. Until then, I'll keep an eye on this.

 

[shenmuee]

Waindigo submitted a new resource:

Book Reviews by Waindigo (version 1.0) - Gives your community a new book reviews area with monetization through Amazon Associates

Well, Can you do one movie review version of this script? =)

 

[Gene Steinberg]

Maybe this has been mentioned, but I didn't see it. I had to revise a review from one of our original posters because he put in the 13 digit ISBN rather than the 10 digit one. Why not support both?

 

[lazer]

+1 for a product (software) review conversion of this add-on

 

[gordy]

+1 for a product (software) review conversion of this add-on

+1 here too

 

[Lisa]

Okay, so in the middle of designing a new style and I've discovered that the reviews pull through the messageList css. I've managed to override the book blurb section by using .messageList .book css but can't figure out what css I can use to override it pulling the messageList css onto the actual reviews.

Screenie attached.

 

[Gene Steinberg]

And what, again, about 13 digit ISBNs?

 

[Kintaro]

It works with all the amazon network? de, it, uk etc...

 

[Jon W]

Okay, so in the middle of designing a new style and I've discovered that the reviews pull through the messageList css. I've managed to override the book blurb section by using .messageList .book css but can't figure out what css I can use to override it pulling the messageList css onto the actual reviews.

You could try using a CSS attribute selector like [itemtype=http:// schema.org/Review]

And what, again, about 13 digit ISBNs?

Not sure if it has been pointed out on this thread yet, but I am a commercial add-on developer. This means that, with the exception of bug fixes (which we contribute for free), further development requires a contribution from the community. This would count as a new feature, rather than a bug fix, so please consider making a small donation or requesting a quote by private conversation and contributing directly to this new feature. Thank you.

It works with all the amazon network? de, it, uk etc...

It should do.

 

[Silvan]

oh, i like this. Any plans to turn this into an album / single version (linking to Amazon/ iTunes)?

 

[petertdavis]

If I use the .com I get the following:

View attachment 29674

I get this same bug.

 

[petertdavis]

A nice enhancement to this would be to tie which Amazon site it uses (.com, .co.uk, etc...) to the user instead of the site as a whole. Most of us have multinational audiences, and if we have members who are going to use the reviews, or even click through to buy the books, I'm sure they'd appreciate the thoughtful touch of sending them to the right store.

 

[Anthony Parsons]

Okay, so in the middle of designing a new style and I've discovered that the reviews pull through the messageList css.

My advice would be to remove:

                        <xen:if is="{$book.book_purchaseurl} == ''">
<iframe style="float:right;height:245px;width:120px;" src="http://rcm-uk.amazon{$amazonDomain}/e/cm?lt1=_blank&bc1=FFFFFF&IS2=1&bg1=FFFFFF&fc1=000000&lc1=0000FF&t={$amazonAssociatesTag}&o=2&p=8&l=as4&m=amazon&f=ifr&ref=ss_til&asins={$book.book_ISBN}" style="width:120px;height:240px;" scrolling="no" marginwidth="0" marginheight="0" frameborder="0"></iframe>
                        </xen:if>

From template: bookreviews_view

There is also another issue in template: bookreviews_review_item

Need to add to the end, after the </li>

<div style="clear:both;"></div>

Because the li's are being floated and not cleared, thus they distort a whole lot when multiple reviews are given. This changes the reviews from:



To:

 

[Lisa]

A problem I've discovered today (I don't know if it's been brought up in the thread so far). When a book is available only as an ebook, there obviously is no ISBN number, which causes you not able to save the book. If you put "none" in the ISBN box it will save, but will not pull through the book image. Would it be possible to issue a fix so that the book image can be pulled via the Kindle code and an option not to put in an ISBN.

 

[Jon W]

A problem I've discovered today (I don't know if it's been brought up in the thread so far). When a book is available only as an ebook, there obviously is no ISBN number, which causes you not able to save the book. If you put "none" in the ISBN box it will save, but will not pull through the book image. Would it be possible to issue a fix so that the book image can be pulled via the Kindle code and an option not to put in an ISBN.

Happy to fix this, but I would not consider this a bug so it would be chargeable. Please send me a private conversation if you would like to contribute.

 

[melbo]

Any way that 'Buy this book' could 'add to cart' rather than just taking one to the page? 'add to cart' comes with a 90 day Amazon affiliate cookie while the link to page sets a 24 hour affiliate cookie.

No big deal but it would be a nice option.