Ok, I’ve made some bug fixes for version 1.2. I considered making this version 1.2.1, however, there were no real changes to the module to warrant a semi-point release. Here’s what’s been fixed:
- Cosmetic changes. (even more) You’ll notice the inline documentation looks much, much prettier
- Fixed a bug with importing OPML files versus OPML urls
- Fixed a bug with PHP 4.4 and passing variables by reference
- Fixed a rather silly bug of not being able to save a linklist because the module thought you were creating a whole new linklist with a duplicate name. :p
Feel free to download the file yet again and make sure you replace all the files (mcp.linklist.php, mod.linklist.php, and lang.linklist.php).
How does this post make you feel?
- Excited
- Fascinated
- Amused
- Bored
- Sad
- Angry
The deeper I’m looking at it the more I appreciate its functionality. Great work and thanks again for your efforts!
But I have one feature request: could wie have {url_added} as a variable and not only as an order option?
Ok, I am probably being an idiot, but whenever LinkList updates on my site, there are PHP warnings at the top of the page. Once the page is refreshed, the warnings are gone, and LinkList *does* fetch updates..just wondering what’s causing the warnings and how to get rid of them.
I seem to have found a strange bug in the EE module side. The site I’m working on has multiple super-admins. Currently, since I created the linklist when I was logged in, I can see and edit and modify the links as I please. However, when another person logs in to the same installation using their profile, the Link List module mentions there’s NO link list created. How can I get it so multiple users can view the linklist module?
the linklist module keeps a linklist for each member their own. The original superadmin (i.e. user number 1) has the ability to view all linklists (but not add). I suppose the functionality to edit existing linklist to all superadmins will be a way to go in this case.
Looks like a functionality for the next version to add in.
Hi, when do you think you’l have the next version completed? :-) Is there a workaround I can add in right now to allow other superadmins to edit and view the list? Thanks!
Installed this for the first time and noticed a few things.
1. recently_updated variable is mentioned under the conditional section of the documentation, but nowhwere else.
2. relative_updated doesn’t seem to work with conditionals. I tried {if relative_updated} and {if relative_updated !- “"} and neither worked.
3. Anything including the count linkcode is not going through my index.php (which I’ve actually named “x"), it’s going directly through my main site. This may have something to do with preferences, however I checked my system prefs and everything’s set up correctly for EE. I’ll poke around in the code later to see if I can figure out why the index page isn’t included in the url construction.
Great job!
Changing all instances of:
$PREFS->core_ini[’site_url’].
to:
$PREFS->core_ini[’site_url’].’/’.$PREFS->core_ini[’site_index’].
seemed to do the job. You might want to look into this for your next bug fix. From what I can see, site_url is your base URL where the index.php resides. I think I could probably solve the problem if I had my “x” page set up as the index for the site, but I don’t think I do, so this may just have been affecting me.
Lynda…was just looking at #3, {if relative_updated}… what are you looking for it to be checking for? it will always be true which is why I’m wondering because it will always display “1 hour ago” or “365 days ago” etc.
Yoshi, I believe I was expecting it to be false or empty for sites that have no RSS feed and therefore cannot possibly have a relative updated time. I’ve found another way to do what I wanted to do, so I understand if a value is returned for the variable no matter what.
yeah, it’s just the way conditionals are done, really. relative_updated is a “custom” variable, so it’s not really defined per se, when in use for a conditional. (since conditionals are parsed before variables are)
If you’ve found an alternate way of doing it, then i’ll leave it as is.