View Issue Details

IDProjectCategoryView StatusLast Update
0002956OXID eShop (all versions)1.01. Products (product, categories, manufacturer, promotions etc.)public2012-12-07 14:28
Reportercoarsy 
PriorityurgentSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.5.0 revision 34568 
Target VersionFixed in Version4.6.0_beta1 
Summary0002956: Article Long Description doesnt inherit from parent article to variant
DescriptionIf an variant has no long description, the long description should inherit
to the variants from the parent article.
Steps To Reproduce1. Login into demoshop backend and choose the anna jeans
2. Go to the variant view of the article and choose one
3. Delete the long description in the variant
4. Go to the frontend and put the former edited article into basket
5. Go into the basket and click back to the main article:
http://demoshop.oxid-esales.com/community-edition/Bekleidung/Fashion/Fuer-Sie/Jeans/Kuyichi-Jeans-ANNA-W-34-L-34-Blau.html
6. There is no article description available anymore
TagsProducts
ThemeBoth
BrowserAll
PHP Versionany
Database Versionany

Relationships

has duplicate 0003516 resolvedLinas Kukulskis oxarticle::getArticleLongDesc() inheritance does not work properly 
has duplicate 0003271 resolvedLinas Kukulskis oxlongdescription is not inherited correctly for subshops 
child of 0003454 resolvedLinas Kukulskis Article Long Description is false 

Activities

coarsy

2011-06-07 09:22

reporter   ~0004702

Last edited: 2011-06-07 09:51

View 2 revisions

Found the error in oxarticle.php at this function:

protected function _assignParentFieldValues()

PHP Code:
if ( $this->getArticleLongDesc()->getRawValue() === null )

should be:
if ( $this->getArticleLongDesc()->getRawValue() == null )

Best regards,

Chris

coarsy

2011-12-08 12:19

reporter   ~0005465

This bug is now back in version 4.5.5.
I've seen, that the getArticleLongDesc was refactorized.

Changing the if ( $sDbValue !== false ) statement
to if ( $sDbValue != false ) is solving this problem.

Are you compliant with me? Would be very great, if you
could provide an hotfix for this issue.

coarsy

2011-12-08 14:06

reporter   ~0005466

Better solution in my opinion is:

if ( $sDbValue !== "" ) {

vilma_liorensaityte

2011-12-13 10:39

reporter   ~0005481

Last edited: 2011-12-13 10:40

View 2 revisions

This problem will be fixed for 4.6.0 release.

coarsy

2011-12-13 13:14

reporter   ~0005483

Could you provide an quickfix or confirm my solution please?
Thanks in advantage!

vilma_liorensaityte

2011-12-13 13:21

reporter   ~0005484

We changed long description loading at all for 4.6.0 version. But your solution to change "if ( $sDbValue !== false)" statment to "if ( $sDbValue != false )" is simmilar.

dainius.bigelis

2011-12-13 13:26

reporter   ~0005486

The offered solution from coarsy will work and is similar to our fix. So the quickfix will not be provided by us.
As Vilma told, our fix will be released together with 4.6.0 version.