CVS edit link for sebas5384

I'm a fan of Drupal CMS, and currently we are building a lot of sites with Drupal.
So would be really great if you can contribute back to the community.

Comments

sebas5384’s picture

Component: Miscellaneous » new project application
Status: Postponed (maintainer needs more info) » Needs review
Issue tags: +blocks, +Context, +user experience, +Drag and Drop
StatusFileSize
new2.79 KB

I have many sites that need a context for some nodes, so the user can drag&drop a block to a individual node.
But was only possible with some steps:
1. Create the Node.
2. Get the NID or the URL of that Node.
3. Create a Context that aplies those conditions.
4. Go back to the node view.
5. Edit the Context adding some blocks, etc... for your Node.

This was to bad experience for users, so I made a module to get a context to you node in an easy way:
1. Create a Node with the option of "Create a Context for this Node" checked. (default content type enabled is Page)
2. Now do the step 5 of the old way. ;)

But, maybe you are thinking but now i'm going to have a huge list of Contexts on the "Context UI", well, no, the contexts generated by this module have a flag telling to hide, so your list of context maintains clean and fast.
You also can choose what content types you want to enabled this feature.

Well with more less, Im leaving the module attached.
I dont't kown what more i have to do to create a project page on the Drupal.org/project, so if someone can help me (if this module is reviewed and approved, of course).

Sebas.

sebas5384’s picture

StatusFileSize
new8.54 KB

Sorry i forgot to commit in my svn after export the module directory :)

SO PLEASE DOWNLOAD THIS ONE !!!

avpaderno’s picture

Status: Needs review » Needs work
Issue tags: -blocks, -Context, -user experience, -Drag and Drop +Module review

Hello, and thank you for applying for a CVS account.

As per requirements, the motivation should include more than two sentences (the exact words are few paragraphs) that describe the module features, and compare the proposed module with the existing solutions.

I will report here a little checklist:

  • The code needs to follow the coding standards; check in particular the format used for the control structures, and the name given to PHP variables, Drupal persistent variables, functions defined from the module.
  • Files available from third-party sites should not be included within the module/theme. This is particularly true for files that are not licensed under GPL License v2, but it is also true for files that are licensed under the same license used by Drupal.
  • The license file should not be included as well; the packaging script already include that file. In any cases, the code for modules/themes committed in drupal.org repository needs to be released under the same license used by Drupal; any compatible license is not allowed.
  • Check the code passes the Coder validation.
sebas5384’s picture

Component: new project application » co-maintainer application
Status: Needs work » Needs review
Issue tags: -Module review +blocks, +Context, +user experience, +Drag and Drop
StatusFileSize
new8.61 KB

Another version with some warning issues fixed.

avpaderno’s picture

Component: co-maintainer application » new project application
Status: Needs review » Needs work
Issue tags: -blocks, -Context, -user experience, -Drag and Drop +Module review
sebas5384’s picture

Component: new project application » co-maintainer application
Status: Needs work » Needs review
Issue tags: -Module review +blocks, +Context, +user experience, +Drag and Drop
StatusFileSize
new10.28 KB

Another and Last Version till the final review :)

sebas5384’s picture

Title: sebas5384 [sebas5384] » Context by Node [sebas5384]
avpaderno’s picture

Title: Context by Node [sebas5384] » sebas5384 [sebas5384]
Component: co-maintainer application » new project application
Issue tags: -blocks, -Context, -user experience, -Drag and Drop +Module review

If we are playing some kind of game, I am sorry but I win. :-)

Please change only the status of the report, when you add an updated version of the module.

avpaderno’s picture

Status: Needs review » Needs work

See also my comment #3, which has been left without any answer.

sebas5384’s picture

StatusFileSize
new5.24 KB

kiamlaluno i'm sorry, i never contributed code with burocratic things, but i'm 100% agree that must have this process.
So please, i ask you to be more patient, i'm just trying to help :)

Well, i make a test with the Coder module and all pass, so, please, give it a try with this new version.
The scope of this module was explained in the comment #1, but maybe i must do a more short and explained text about the module.

Thanks!

sebas5384’s picture

Status: Needs work » Needs review
matglas86’s picture

StatusFileSize
new3 KB

So basically what this module does is create a context specified for 1 node based on its NID. When the node is created its possible to edit the context.

First I did not succeed to edit the context because it was made invisible. That was for good reason I remember out of our IRC discussion. When a lot of nodes create a context it would get a little to much. Now what I did was add a link to the context by node fieldset on the edit form of the node. This makes it possible to go to the context from the node.

Also I cleaned up some tabbing in the code that is against coding standards.

sebas5384’s picture

StatusFileSize
new5.35 KB

Thanks matglas86 !!

I made some other changes and now its final :)

So please download this version if you are going to review, use it or what ever.

Sebas.

matglas86’s picture

@sebas5384 I'll take a look at it but please when you post a new update give a short explanation about what you changed. Now I should go a look if I see the change? So please leave a post about your changes.

zzolo’s picture

Status: Needs review » Postponed

Hi. Please read all the following and the links provided as this is very important information about your CVS Application:

Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for your application. Please read the following documentation on how this affects and benefits you and the application process:
Migrating from CVS Applications to (Git) Full Project Applications

  • The status of this application will be put to "postponed" and by following the instructions in the above link, you will be able to reopen it.
  • Or if your application has been "needs work" for more than 5 weeks, your application will be marked as "closed (won't fix)". You can still reopen it, by reading the instructions above.
avpaderno’s picture

Issue summary: View changes
Status: Postponed » Closed (won't fix)

As per previous comment, I am setting this issue to won't fix.

Since new users can now create full projects, applications have a different purpose and they are handled on a different issue queue. See Apply for permission to opt into security advisory coverage for more information.