gbp_permanent_links

pieman · 2009-11-03 10:38:38

pieman wrote:

Used in the News section, this returns the gbp_pl defined (section/yyyy/mm/dd/title) url:
<txp:article_custom section="news"><a href="<txp:permlink />" rel="bookmark"><txp:title /></a></txp:article_custom>
But used in any other section, exactly the same code returns a /section/title url instead – even though I have prefs=messy (!)

Graeme

With fresh eyes this morning I saw that the culprit wasn’t gbp_pl, but in fact me (shome mishtayke shurely).

I was trying to be clever and pointing tru_tags to the News section and somehow that was sinking the ship. All is well now. Apologies for the false alarm.

maniqui · 2009-11-03 12:37:31

Ops. I wrote a post last night, but it seems I forgot to submit it.

Is there any chance that, after upgrading, all the gbp_p_l rules have been deleted by the upgrade process?

I upgraded, but didn’t check the gbp_p_l tab until a few hours later. I was surprised to see that there weren’t any rules, like if they were deleted.
FYI, I run gbp_p_l from the plugin_cache_dir, and to upgraded I simple switched old version file for new one.

I wonder if it was gbp_p_l the culprit of this deletion. Next time I’ll backup :)

graeme · 2009-11-03 13:08:15

maniqui wrote:

I wonder if it was gbp_p_l the culprit of this deletion. Next time I’ll backup :)

No changes were made to the plugin which would of caused the deletion. It’s very strange. the plugin stores it’s setting in the txp_prefs table so you may want to check your other settings to make sure everything is still set up correctly.

maniqui · 2009-11-03 13:16:45

While here, a minor wish:

If you generate the “typical” /section/category/title permlink, when you visit an article permlink (/my-section/the-category/my-great-article), the txp:if_category tag will return true for the “page context”, which I consider an “erroneous behaviour”.

I think it would be “in harmony” with standard TXP behaviour [1] if it would return false, as we are not on a category list (article_list context), but already visiting the permlink of an article (individual_article context).

Of course, if this change is possible and it’s applied, it may break backwards compatibility. I won’t worry about that, anyway! :D

[1] From if_category Textbook entry: “The tag will execute the contained statements if the name attribute matches a category search value, or the list is an article list by category (?c=category) “

maniqui · 2009-11-03 13:57:07

OK, after a quick chat with graeme, he made me notice that that’s how thing work on the inner cogs of TXP, because:

because /section/category/title effectively means /?s=section&c=category&id=someid

And also:

/section/title?c=category is valid (returns true)

Even if the above URLs may not match any real use case, that’s how things works on the background. TXP evaluates if the &c=name ($c variable) is set, and so, if_category returns true, even if on an individual article.

The possible “workarounds”:

test for if_individual_article instead of testing for if_category
test for if_article_list first and then nest a if_category (aka “if_category_list” tag, yet to be invented).

Thanks, graeme!

uli · 2009-11-03 23:30:32

Thanks for letting us know, Julián!

aswihart · 2009-11-08 22:26:56

One issue I’m having for end-users, not sure if it’s specific to Txp 4.2: even wtih Freelancer privileges, I can see and edit the settings of this plugin in the Admin tab.

uli · 2009-11-08 23:07:09

Funny I never remarked that. Thanks for the heads-up!

~~In the gbp library look for:~~
~~@var $permissions = ’1,2,3,4,5,6’;@ (two instances)~~
~~change to the user levels that you need there.~~

See graeme’s post

Last edited by uli (2009-11-10 14:49:49)

aswihart · 2009-11-10 02:16:49

Thanks mate! Yeah, I think it’s probably prudent to make it default to at least 1,2,3, if not just 1!?

graeme · 2009-11-10 14:11:24

uli wrote:

In the gbp library look for:
var $permissions = '1,2,3,4,5,6'; (two instances)
change to the user levels that you need there.

Instead of modifying the admin library can I suggest you define var $permissions = etc... in the PermanentLinks, PermanentLinksBuildTabView and PermanentLinksListTabView classes within the gbp_pl source. The admin library is used for other plugins (eg. MLP) so any changes made to it would also effect them too.

uli · 2009-11-10 14:27:29

graeme wrote:

The admin library is used for other plugins (eg. MLP) so any changes made to it would also effect them too.

Ouch! Thanks

aswihart · 2009-11-14 17:13:28

Any tips on getting this plugin to play nice with jmd_rate? See my issues in that plugin’s thread.

Textpattern CMS

Textpattern CMS support forum

#517 2009-11-03 10:38:38

Re: gbp_permanent_links

#518 2009-11-03 12:37:31

Re: gbp_permanent_links

#519 2009-11-03 13:08:15

Re: gbp_permanent_links

#520 2009-11-03 13:16:45

Re: gbp_permanent_links

#521 2009-11-03 13:57:07

Re: gbp_permanent_links

#522 2009-11-03 23:30:32

Re: gbp_permanent_links

#523 2009-11-08 22:26:56

Re: gbp_permanent_links

#524 2009-11-08 23:07:09

Re: gbp_permanent_links

#525 2009-11-10 02:16:49

Re: gbp_permanent_links

#526 2009-11-10 14:11:24

Re: gbp_permanent_links

#527 2009-11-10 14:27:29

Re: gbp_permanent_links

#528 2009-11-14 17:13:28

Re: gbp_permanent_links

Board footer