Sign in Register
Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at


  • keilo 568 posts 1023 karma points
    May 12, 2015 @ 19:39
    keilo
    0

    Thanks!

    uSEO PRO

    Thanks for sharing this, very complete and equally useful.

    Couple of suggestions for improvement would be; 

    a) allow walking up the tree if no seofields defined on the currentpage, by using CurrentPage._seoFields

    if seofields not field on given page (say news/events page where editor dont usually fill each and every single one) it throws error such as

    'string' does not contain a definition for 'noseo'

    Line 9:      {

    Line 10:

    Line 11:         if(seo.noseo == null || seo.noseo == false)

    Line 12:

    Line 13:         {

    b) allow the "Static Title" to be selectable in the property - for completeness

     

    Cheers!

    Copy Link
  • Luke 110 posts 256 karma points
    May 13, 2015 @ 08:56
    Luke
    1

    Hi Keilo,

    Thank you so much for trying and for the great feedback. I will change the example code as per your suggestions to:

    var seo = CurrentPage._seoFields;

if(!IsNullOrEmpty(seo))

    This removes the error that you were getting.

    Lastly - "Static Title" I was not and I am still not sure how to implement this. When it comes to me I will ammend :-)

    Thanks once again :-)

    Luke

    Copy Link
Please Sign in or register to post replies

Write your reply to:

Draft