New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error uploading article page image #1

Closed
barisakpunar opened this Issue Aug 28, 2015 · 7 comments

Comments

Projects
None yet
3 participants
@barisakpunar

Server Error in '/' Application.

Property TopImage not found!

Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.

Exception Details: System.Exception: Property TopImage not found!

Source Error:

An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.

Stack Trace:

[Exception: Property TopImage not found!]
KalikoCMS.Core.Collections.PropertyCollection.set_Item(String propertyName, PropertyData value) +198
KalikoCMS.Core.EditablePage.SetProperty(String propertyName, PropertyData value) +83
KalikoCMS.Admin.Content.EditPage.SavePropertiesForPage(EditablePage editablePage) +1393
KalikoCMS.Admin.Content.EditPage.SaveDataForNewPage() +205
KalikoCMS.Admin.Content.EditPage.SaveData() +194
KalikoCMS.Admin.Content.EditPage.PublishButtonEventHandler(Object sender, EventArgs e) +104
System.Web.UI.WebControls.LinkButton.RaisePostBackEvent(String eventArgument) +153
System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint) +3804

Version Information: Microsoft .NET Framework Version:4.0.30319; ASP.NET Version:4.0.30319.34249

@fschultz

This comment has been minimized.

Show comment
Hide comment
@fschultz

fschultz Dec 16, 2015

Contributor

I'm very sorry I haven't answered this before now. There seems to been something wrong with the issue notification setup :(

Just wanted to check if this worked out or if there's anything I can help with?

Contributor

fschultz commented Dec 16, 2015

I'm very sorry I haven't answered this before now. There seems to been something wrong with the issue notification setup :(

Just wanted to check if this worked out or if there's anything I can help with?

@barisakpunar

This comment has been minimized.

Show comment
Hide comment
@barisakpunar

barisakpunar Dec 17, 2015

Thank you. I have solved this problem by changing the property name different. The uppercase "I" in the property name was making problem. As I remember when system was converting it to lowercase letters property names was not matching. If you need I can try to reproduce error.

Thank you. I have solved this problem by changing the property name different. The uppercase "I" in the property name was making problem. As I remember when system was converting it to lowercase letters property names was not matching. If you need I can try to reproduce error.

@fschultz

This comment has been minimized.

Show comment
Hide comment
@fschultz

fschultz Dec 28, 2015

Contributor

If you have the time that would be much appreciated, thanks!

Contributor

fschultz commented Dec 28, 2015

If you have the time that would be much appreciated, thanks!

@barisakpunar

This comment has been minimized.

Show comment
Hide comment
@barisakpunar

barisakpunar Dec 29, 2015

The problem is as follows.

In PropertyCollection class when it is matching the property items method uses propertyName.ToLowerInvariant() to match.

internal PropertyItem GetItem(string propertyName) {          
        return Properties.Find(p => p.PropertyName == propertyName.ToLowerInvariant());
    }

But normally properties are filled in PropertyData class with using ToLower()

select new PropertyItem {
                        PagePropertyId = m.PagePropertyId,
                        PropertyName = p.Name.ToLower(),
                        PropertyData = CreatePropertyData(p.PropertyTypeId, m.PageData),
                        PropertyId = p.PropertyId,
                        PropertyTypeId = p.PropertyTypeId
                    };

In the TopImage property
İn my computer ToLowerInvariant() returns "topimage"
But ToLower() returns "topımage" with no dot over 'i

So they dont match with ant throws an error " Additional information: Property TopImage not found!"

The problem is as follows.

In PropertyCollection class when it is matching the property items method uses propertyName.ToLowerInvariant() to match.

internal PropertyItem GetItem(string propertyName) {          
        return Properties.Find(p => p.PropertyName == propertyName.ToLowerInvariant());
    }

But normally properties are filled in PropertyData class with using ToLower()

select new PropertyItem {
                        PagePropertyId = m.PagePropertyId,
                        PropertyName = p.Name.ToLower(),
                        PropertyData = CreatePropertyData(p.PropertyTypeId, m.PageData),
                        PropertyId = p.PropertyId,
                        PropertyTypeId = p.PropertyTypeId
                    };

In the TopImage property
İn my computer ToLowerInvariant() returns "topimage"
But ToLower() returns "topımage" with no dot over 'i

So they dont match with ant throws an error " Additional information: Property TopImage not found!"

@fschultz

This comment has been minimized.

Show comment
Hide comment
@fschultz

fschultz Dec 30, 2015

Contributor

Thank you very much for getting my attention to this problem and the descriptive bug report!

I will go through the complete project to ensure that the invariant version of the function is used so that strings are converted in a culture independent manner also in other parts of the system.

Contributor

fschultz commented Dec 30, 2015

Thank you very much for getting my attention to this problem and the descriptive bug report!

I will go through the complete project to ensure that the invariant version of the function is used so that strings are converted in a culture independent manner also in other parts of the system.

@fschultz fschultz added the bug label Dec 30, 2015

@fschultz fschultz self-assigned this Dec 30, 2015

@fschultz

This comment has been minimized.

Show comment
Hide comment
@fschultz

fschultz Jan 20, 2016

Contributor

Version 1.1.1 just released should fix this issue.

Contributor

fschultz commented Jan 20, 2016

Version 1.1.1 just released should fix this issue.

@Matthew-Bonner

This comment has been minimized.

Show comment
Hide comment
@Matthew-Bonner

Matthew-Bonner Aug 17, 2017

Please close, this is no longer an issue and it was fixed in January last year.

Please close, this is no longer an issue and it was fixed in January last year.

@fschultz fschultz closed this Aug 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment