[drupal-devel] [feature] Prefill 'path' field on new node form
eafarris
drupal-devel at drupal.org
Wed Jul 27 18:50:59 UTC 2005
Issue status update for
http://drupal.org/node/27155
Post a follow up:
http://drupal.org/project/comments/add/27155
Project: Drupal
Version: cvs
Component: path.module
Category: feature requests
Priority: normal
Assigned to: Anonymous
Reported by: jakeg
Updated by: eafarris
Status: patch
Attachment: http://drupal.org/files/issues/node.module.fill.node.with.get.patch (497 bytes)
/me gets errors with your patch, but here's a tested version. I really
like this idea; without it, I can't get a bookmarklet to fill in the
'url' field of the weblink node type, for example.
eafarris
Previous comments:
------------------------------------------------------------------------
Wed, 20 Jul 2005 17:43:52 +0000 : jakeg
I'm creating wiki-like features on my site so that if someone with the
permission to create new pages goes to a 'page not found' page then it
gives them an option to create a page at that address (i.e. at that
path, so if they goto /somepath they can easily create a page with path
/somepath).
To do this its very handy for the 'path alias' field on the new node
page to be pre-filled with a value using e.g.
node/add/page?edit[path]=somepath in the same way that you can prefill
the title field using node/add/page?edit[title]=sometitle
I've got this to work by adding the following line to path.module at
line 211 as thus:
case 'form pre':
if(!$node->path AND isset($_GET['edit']['path']))
$node->path = $_GET['edit']['path'];
$output = ...
... my line is the 'if' line in the middle above.
Could this be added to core? I'm sure others could find it handy and it
would save me from having to rewrite the line on every upgrade. If it
can't be added to core, is there anyway for me to put this line into my
own custom module somehow so that I don't have to rewrite it every time
I upgrade?
------------------------------------------------------------------------
Wed, 27 Jul 2005 16:18:31 +0000 : leafish_paul
Not a patch, but I quite like the idea.
------------------------------------------------------------------------
Wed, 27 Jul 2005 16:22:09 +0000 : leafish_paul
Attachment: http://drupal.org/files/issues/path.module.wiki.style.create.page.at.404.patch (956 bytes)
Here you go, ready for voting.
------------------------------------------------------------------------
Wed, 27 Jul 2005 17:40:16 +0000 : Bèr Kessels
Hmm. I am not sure. In this way, it can only be used by the wiki module.
And IMO it is a bad thing to facilitate single implementations in core.
In other words: I would definitely +1 this were a more general
solution. So, what about a hook? IMNSHO that is actually over the top,
a hook, but it would help other modules that want to fiddle with the
path too.
------------------------------------------------------------------------
Wed, 27 Jul 2005 17:54:27 +0000 : jakeg
It shouldn't be implemented as a hook, that is overkill. Apart from wiki
modules, you may generally want to be able to submit a node with any
number of the fields prefilled (i.e. not just 'path' but other fields
as well... though I can't think of an example right now). Having lines
such as the one I suggest for those fields is the only way to make this
possible without individuals having to hack the core and update their
hacks each time there's a new release.
------------------------------------------------------------------------
Wed, 27 Jul 2005 18:14:36 +0000 : clydefrog
jakeg, an example is book module, where the "add child page" link uses
its own URL syntax to prefill the parent field. If a hook for listing
prefillable fields existed, the URL syntax could be made consistent
with the rest of core and contrib.
------------------------------------------------------------------------
Wed, 27 Jul 2005 18:17:22 +0000 : Bèr Kessels
an overkill indeed.
but $node->path = $_GET['edit']['path'] just doesn't feel right. So I
was thinking aloud about how to solve this. Anyone?
------------------------------------------------------------------------
Wed, 27 Jul 2005 18:27:42 +0000 : clydefrog
Attachment: http://drupal.org/files/issues/node-prefill.patch (1023 bytes)
Why not just prefill $node with everything in $_GET['edit']? This patch
should do that (untested)
More information about the drupal-devel
mailing list